Skip to content

Conversation

@aqrln
Copy link
Member

@aqrln aqrln commented Aug 27, 2024

Follow-up to #4986.

  • Change the build.rs scripts to pass through the GIT_HASH environment variable if it's already set.
  • Set dummmy GIT_HASH value in the Nix packages used in the engines size dashboard.
  • Consistently use the new logic everywhere.

Fixes: #4991
Closes: https://github.com/prisma/team-orm/issues/1261

Follow-up to #4986.

* Change the `build.rs` scripts to pass through the `GIT_HASH`
  environment variable if it's already set.

* Set dummmy `GIT_HASH` value in the Nix packages used in the
  engines size dashboard.

* Consistently use the new logic everywhere.

Fixes: #4991
Closes: prisma/team-orm#1261
@aqrln aqrln requested a review from a team as a code owner August 27, 2024 15:26
@aqrln aqrln requested review from SevInf and jkomyno and removed request for a team August 27, 2024 15:26
@@ -1,13 +1,4 @@
extern crate cbindgen;
Copy link
Member Author

Choose a reason for hiding this comment

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

extern crate removal is unrelated, opportunistic cleanup

@codspeed-hq
Copy link

codspeed-hq bot commented Aug 27, 2024

CodSpeed Performance Report

Merging #4992 will not alter performance

Comparing git-hash-nix (c019124) with main (5fe2181)

Summary

✅ 11 untouched benchmarks

@github-actions
Copy link
Contributor

WASM Query Engine file Size

Engine This PR Base branch Diff
Postgres 2.063MiB 2.063MiB -1.000B
Postgres (gzip) 823.717KiB 823.741KiB -24.000B
Mysql 2.033MiB 2.033MiB 2.000B
Mysql (gzip) 811.294KiB 811.290KiB 5.000B
Sqlite 1.924MiB 1.924MiB 2.000B
Sqlite (gzip) 768.301KiB 768.297KiB 4.000B

@aqrln aqrln added this to the 5.20.0 milestone Aug 27, 2024
@jkomyno jkomyno merged commit 8225295 into main Aug 28, 2024
@jkomyno jkomyno deleted the git-hash-nix branch August 28, 2024 09:38
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.

Fix broken engine size publishing workflow

3 participants