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

chore: automated tests for bundled core in apex-node and templates #1070

Merged
merged 14 commits into from
Jun 3, 2024

Conversation

CristiCanizales
Copy link
Contributor

@CristiCanizales CristiCanizales commented May 15, 2024

What does this PR do?

  • Creates new workflows testCoreBundling.yml and testExternalProjects.yml to automate testing for core bundled in apex-node and templates repo
  • Extracts bundling step from esbuild-publish.yml into new workflow bundle.yml
  • Automates core-bundle release for every new core github release

What issues does this PR fix or reference?

@W-15634761@

Test run: https://github.com/forcedotcom/sfdx-core/actions/runs/9341655590/job/25708669666?pr=1070

@CristiCanizales CristiCanizales self-assigned this May 15, 2024
@CristiCanizales CristiCanizales marked this pull request as ready for review May 15, 2024 23:12
@CristiCanizales CristiCanizales requested a review from a team as a code owner May 15, 2024 23:12
@CristiCanizales
Copy link
Contributor Author

Copy link
Member

@mingxuanzhangsfdx mingxuanzhangsfdx left a comment

Choose a reason for hiding this comment

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

  1. fix the bundle and publish workflow. In your current change, the publishing jbo does not wait the bundling job, and they are on separate workspaces actually.
  2. figure out why there are 2 test failures in templates on windows.
  3. figure out why prdNuts fails, might need to bump the dependencies and also reach out to CLI when necessary.
  4. several questions I left.

@mingxuanzhangsfdx
Copy link
Member

Remember to automate the release of core-bundle as well(I just updated in the AC of the work item)

CristiCanizales and others added 2 commits June 3, 2024 15:18
build executed only when the previous workflow succeeds or manually
Copy link
Member

@WillieRuemmele WillieRuemmele left a comment

Choose a reason for hiding this comment

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

I'm no GHA wizard - but these all look accurate to me

@WillieRuemmele WillieRuemmele merged commit 6fdf8a2 into main Jun 3, 2024
73 checks passed
@WillieRuemmele WillieRuemmele deleted the cristi/w-15634761 branch June 3, 2024 21:00
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.

4 participants