Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ER 782 Create app service to host web application dev subscription #653

Conversation

ebrett
Copy link
Contributor

@ebrett ebrett commented Oct 3, 2023

Ticket and context

[ER-782] Get the webapp up

@github-actions
Copy link

github-actions bot commented Oct 3, 2023

@ebrett ebrett force-pushed the ER-782-create-app-service-to-host-web-application-dev-subscription branch 10 times, most recently from 57c14dc to 11fcd94 Compare October 3, 2023 15:16
ebrett added 2 commits October 3, 2023 16:32
- terraform azure network
- terraform azure database
- terraform azure web
@ebrett ebrett force-pushed the ER-782-create-app-service-to-host-web-application-dev-subscription branch from 11fcd94 to d91b141 Compare October 3, 2023 15:33
push:
branches:
- main
paths-ignore:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The array of files to ignore might want looking at. Some of these files don't exist in this project and should be removed but others might also need to be added in their place.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Agree on this point.

Copy link
Contributor Author

@ebrett ebrett Oct 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did look at them and removed the ones below - the ones I kept were either there or could be added but should probably still be ignored.

  - .pa11yci
  - uml/*

@ebrett ebrett added this pull request to the merge queue Oct 4, 2023
Merged via the queue into main with commit 03a8e63 Oct 4, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants