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.

Sign up for GitHub

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

Jump to bottom

WIP #818

Open
wants to merge 82 commits into
base: main
Choose a base branch
from
Open

WIP #818

wants to merge 82 commits into from

Conversation

doshitan
Copy link
Contributor

Ticket

Resolves #{TICKET NUMBER OR URL}

Changes

What was added, updated, or removed in this PR.

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers.

Testing

Provide evidence that the code works as expected. Explain what was done for testing and the results of the test plan. Include screenshots, GIF demos, shell commands or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

lorenyu and others added 30 commits October 3, 2024 09:44
@lorenyu
Add copier files
8aee09e
@lorenyu
Templatize app_name
a44b003
@lorenyu
Rename e2e/app to e2e/{{app_name}}
2811ba0
@lorenyu
Merge branch 'main' into lorenyu/platform-cli
f7661c4
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
3f649ef
@doshitan
Update cd-{{app_name}}.yml to parameterize job name
10eea5b
@doshitan
Parameterize workflows/ci-{{app_name}}* content for app_name
37cc0d1
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
c1aa4cb
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
58bd26f
@doshitan
WIP try local port setting
11076e5
@doshitan
Have app vuln scans run on any changes to underlying action (as was o…
1120558 
…riginal intention I believe)
@doshitan
Support conditional blocks based on if dev env is available
9252220
@doshitan
Make infra service checks per-app?
0097c5b
@doshitan
Dummy change for updating
fcff0d1
@doshitan
Use /health endpoint for check in infra_test.go
0733d96
@doshitan
Disable TLS checks in infra_test.go
f2d5e34
@doshitan
Update docs for broken links and consistently use <APP_NAME>
dd8a1c7
@doshitan
Update decision docs for template usage
a023ba3
@doshitan
More broken doc link updates
81f514d
@doshitan
More broken doc link updates
37a2962
@doshitan
Update template-infra's infra check to install example project to run…
b6f626e 
… against
@doshitan
Merge branch 'main' into lorenyu/platform-cli
46a4114 
* main:
  Update obsolete GitHub Actions (#774)
@doshitan
Pull infra/project-config/main.tf into template
25904e1
@doshitan
Fix doc link lint due to project-config templating
4ad924a
@doshitan
Fix IMAGE_TAG when not in a git repo
f968f02 
Need to define DATE before using it.
@doshitan
template_infra_test.go: Build the IMAGE_TAG specified
c44ae5a
@doshitan
bin/check-github-actions-auth: be explicit about the repo to check
536a572
@doshitan
template_infra_test.go: comment on not using /health
2e39545
@doshitan
Prefix all copier variables with template part name (base or app)
c53219c
@doshitan
Only ask relevant questions for different template
5ab08b3 
The CLI tool will inject the `template` answer, so generally the end
users will never see/answer it directly.
coilysiren and others added 13 commits November 27, 2024 15:19
@coilysiren
remove feature flags due to evidently deprecation (part 2)
ccc8cab
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
7053f3d
@coilysiren
resolve merge conflicts against main
697aac4
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
383a253
@doshitan
Build out docs/release.md a little bit
2693314
@doshitan
Exclude template-only files that aren't tagged as such in the name
494f7c3 
By specifying `_exclude` in `copier.yml`, we need to re-add the relevant
default excludes to the list (`.git` and `copier.yml`).

https://copier.readthedocs.io/en/stable/configuring/#exclude
@doshitan
Remove old install/update scripts
82ec64a
@doshitan
Fix destroy-pr-environment for multi-app PR env setup
e61b207
@doshitan
e2e doc fixes/consistency
88164a5
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
035ef09
@nava-platform-bot
Merge remote-tracking branch 'origin/main' into lorenyu/platform-cli
43b79c6
@lorenyu
Merge branch 'main' into lorenyu/platform-cli
68a122d
@doshitan
Fix custom setup-terraform usage in template-only-ci-infra.yml
7fb0495
@doshitan doshitan force-pushed the doshitan/platform-cli branch from 3f83e98 to de1e80f Compare January 2, 2025 15:37
@doshitan doshitan force-pushed the doshitan/platform-cli branch 2 times, most recently from d6f9c86 to abc854e Compare January 3, 2025 16:19
@doshitan doshitan changed the base branch from main to lorenyu/platform-cli January 3, 2025 22:04
@doshitan doshitan force-pushed the doshitan/platform-cli branch from cfb91c4 to 3e148a7 Compare January 6, 2025 18:15
Base automatically changed from lorenyu/platform-cli to main January 14, 2025 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@doshitan @lorenyu @nava-platform-bot @coilysiren