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(ci): ensure that the artifact rebuilding PR will run CI #2824

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

TomAFrench
Copy link
Member

Description

Problem*

Resolves

Summary*

Currently the PR to automatically update the build artifacts is being committed by the github-actions bot but this doesn't allow for workflows to run on the created PR. A workaround for this is for us to make the PR as @kevaundray which will then run CI as usual.

The alternative for this is that we have to always close and reopen the PR before CI will run.

Documentation

  • This PR requires documentation updates when merged.

    • I will submit a noir-lang/docs PR.
    • I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@kevaundray kevaundray added this pull request to the merge queue Sep 25, 2023
Merged via the queue into master with commit bfaff82 Sep 25, 2023
26 checks passed
@kevaundray kevaundray deleted the tf/fix-auto-build-artifacts branch September 25, 2023 21:54
TomAFrench added a commit that referenced this pull request Sep 26, 2023
* master: (31 commits)
  fix: Conditionally run the "Create or Update PR" step in acir artifacts rebuild workflow (#2849)
  fix: lack of cjs package version (#2848)
  chore: delete unnecessary nix files (#2840)
  chore!: `generateWitness` now returns a serialized witness file (#2842)
  chore: fix `acvm_js` linting and tests (#2834)
  feat: remove redundant predicate from brillig quotients (#2784)
  chore: Fix dependencies in acvm_js and remove local lock file (#2833)
  chore: add acvm_js to integration build
  chore: update discord link (#2831)
  chore: discard changes in nargo directory
  chore: Update ACIR artifacts (#2810)
  chore: yarn install
  chore: Remove acvm tracking workflow (#2829)
  add acvm_js to workspace
  - remove acvm-eslint - rename acvm-test to acvm-js-test
  chore: delete acvm-release workflow
  chore: merge flake.nix files and remove inner workspace (#2826)
  chore(ci): remove stale job steps (#2816)
  chore(ci): ensure that the artifact rebuilding PR will run CI (#2824)
  feat(traits): Implement trait bounds typechecker + monomorphizer passes (#2717)
  ...
Sakapoi pushed a commit to Sakapoi/noir_fork that referenced this pull request Oct 19, 2023
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.

2 participants