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

Move non-vendored dependencies build out of main CI workflow #3755

Closed
hs-apotell opened this issue Jul 14, 2023 · 1 comment
Closed

Move non-vendored dependencies build out of main CI workflow #3755

hs-apotell opened this issue Jul 14, 2023 · 1 comment
Assignees
@hs-apotell

Comments

@hs-apotell
Copy link
Collaborator

This configuration fails anytime the UHDM or Surelog API changes, preventing from regression run. No regression run means no logs.
Version update can only happen on master branch i.e. after a PR is merged. So, until UHDM PR is merged, Surelog builds cannot be verified.

Two potential options -

  • Move it to a nightly verification build
  • Make it a separate job that doesn't block regression runs

@hzeller @timkpaine
hs-apotell added a commit to Apotell/Surelog that referenced this issue Jul 27, 2023
@hs-apotell
Issue chipsalliance#3755: Move non-vendored workflow out of main
29ea0ba 
Move non-vendored workflow out of main and into a separate workflow.
Also, run it only for master branch, this is where it matters.
hs-apotell added a commit to Apotell/Surelog that referenced this issue Jul 28, 2023
@hs-apotell
Issue chipsalliance#3755: Move non-vendored workflow out of main
5738d86 
Move non-vendored workflow out of main and into a separate workflow.
Also, run it only for master branch, this is where it matters.
@hs-apotell hs-apotell mentioned this issue Jul 28, 2023
Merged
@hs-apotell hs-apotell self-assigned this Jul 28, 2023
alaindargelas added a commit that referenced this issue Jul 28, 2023
@alaindargelas
Merge pull request #3779 from Apotell/Issue#3755
b38213f 
Issue #3755: Move non-vendored workflow out of main
@hs-apotell
Copy link
Collaborator Author

Fixed by #3779

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hs-apotell hs-apotell

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hs-apotell