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

feat: allow tracing build system with [debug ci] #2389

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

ludamad
Copy link
Collaborator

@ludamad ludamad commented Sep 19, 2023

With this PR you are now able to put [debug ci] into a commit message and have all BASH scripts set -x, which prints all commands ran.

Going forward, I encourage checking if [ -n "${BUILD_SYSTEM_DEBUG:-}" ] ; then ... fi with verbose debug dumps of data.

Changes:

  • setup_env exports this flag
  • all other bash scripts in build-system or in e.g. scripts/run_tests module locations use this flag

@ludamad ludamad force-pushed the ad/feat/debug-ci-message branch 3 times, most recently from a52cfd1 to 476d7fc Compare September 19, 2023 08:37
@ludamad ludamad force-pushed the ad/feat/debug-ci-message branch from 476d7fc to e8e5fb5 Compare September 19, 2023 08:39
Copy link
Member

@Maddiaa0 Maddiaa0 left a comment

Choose a reason for hiding this comment

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

most groovy

@Maddiaa0 Maddiaa0 enabled auto-merge (squash) September 19, 2023 08:47
@Maddiaa0 Maddiaa0 merged commit ce311a9 into master Sep 19, 2023
2 checks passed
@Maddiaa0 Maddiaa0 deleted the ad/feat/debug-ci-message branch September 19, 2023 08:51
PhilWindle pushed a commit that referenced this pull request Sep 19, 2023
🤖 I have created a release *beep* *boop*
---


<details><summary>aztec-packages: 0.7.8</summary>

##
[0.7.8](aztec-packages-v0.7.7...aztec-packages-v0.7.8)
(2023-09-19)


### Features

* `NodeInfo` cleanup
([#2370](#2370))
([89fe978](89fe978))
* Allow custom ports in sandbox
([#2393](#2393))
([41ef378](41ef378))
* Allow tracing build system with [debug ci]
([#2389](#2389))
([ce311a9](ce311a9))
* **docs:** Show current noir version for aztec in docs
([#2379](#2379))
([5c7b2ab](5c7b2ab))


### Bug Fixes

* Build script exiting on failed grep
([#2384](#2384))
([e70a781](e70a781))
* Bump e2e_sandbox_example.test.ts timeout
([#2391](#2391))
([9a1bb62](9a1bb62))
* Compile script for the unboxed project
([#2380](#2380))
([2801da2](2801da2))
* Force_deploy_build error
([#2375](#2375))
([4d1cbf9](4d1cbf9))
* Update aztec sandbox getting started markdown
([#2374](#2374))
([a3c6bcf](a3c6bcf))


### Miscellaneous

* Adds on-brand design to private token project
([#2355](#2355))
([072e313](072e313))
* Docs restructure
([#2322](#2322))
([1368b55](1368b55))


### Documentation

* Updated noirup command
([#2339](#2339))
([5308c21](5308c21))
</details>

<details><summary>barretenberg.js: 0.7.8</summary>

##
[0.7.8](barretenberg.js-v0.7.7...barretenberg.js-v0.7.8)
(2023-09-19)


### Miscellaneous

* **barretenberg.js:** Synchronize aztec-packages versions
</details>

<details><summary>barretenberg: 0.7.8</summary>

##
[0.7.8](barretenberg-v0.7.7...barretenberg-v0.7.8)
(2023-09-19)


### Features

* Allow tracing build system with [debug ci]
([#2389](#2389))
([ce311a9](ce311a9))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants