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

ci: revise workflows #1313

Merged
merged 12 commits into from
Nov 13, 2024
Merged

ci: revise workflows #1313

merged 12 commits into from
Nov 13, 2024

Conversation

DerekRoberts
Copy link
Member

@DerekRoberts DerekRoberts commented Nov 13, 2024 •
edited by github-actions bot
Loading

Simplifications:

  • drop NAME parameter from workflows
  • move legacydb in with other builds
  • use PR number through merge.yml
  • move release to end of merge.yml
  • banish schemaspy to scheduled job
  • ensure deployments have random envars (triggers!)
  • replace reusable-doc job with qs-helpers workflow
  • remove coverage comments (sonar cloud incoming)

Thanks for the PR!

Deployments, as required, will be available below:
Any successful deployments (not always required) will be available here

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@DerekRoberts DerekRoberts self-assigned this Nov 13, 2024
@bcgov bcgov deleted a comment from github-actions bot Nov 13, 2024
@github-actions GitHub Actions
Copy link
Contributor

Overall Project NaN% NaN% 🍏

There is no coverage information present for the Files changed

@DerekRoberts DerekRoberts changed the title ci: simplify workflows ci: revise workflows Nov 13, 2024
paulushcgcj
paulushcgcj approved these changes Nov 13, 2024
View reviewed changes
Copy link
Contributor

@paulushcgcj paulushcgcj left a comment

Choose a reason for hiding this comment

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

If is not a lot to ask, can you update the readme file to reflect the new spydoc content folder?

@DerekRoberts DerekRoberts merged commit d6817f7 into main Nov 13, 2024
31 checks passed
@DerekRoberts DerekRoberts deleted the ci/simplify branch November 13, 2024 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mamartinezmejia mamartinezmejia mamartinezmejia approved these changes

@paulushcgcj paulushcgcj paulushcgcj approved these changes

Assignees

@DerekRoberts DerekRoberts

Labels
ci
Projects
DevOps (NR)
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DerekRoberts @paulushcgcj @mamartinezmejia