Skip to content

Conversation

gribnoysup
Copy link
Collaborator

Another thing that I missed while making builds work in GitHub CI: because we are doing a canary release that doesn't commit anything, the version lifecycle is not running and compass doesn't get a dependencies version bump while doing a release. With this change we should finally get a Compass build in CI that is happening against local modules

Otherwise packages/compass doesn't get the version update
@gribnoysup gribnoysup force-pushed the third-time-is-the-charm branch from 39e3a4e to b489dd5 Compare June 20, 2021 10:08
@gribnoysup
Copy link
Collaborator Author

Anticipating this to fail because seems like we now have direct dependency on mongodb-client-encryption somewhere in the Compass production dependencies tree and electron-rebuild fails to rebuild it due to missing mongocryptd headers, it's an optional dep though so will probably add a workaround for now to ignore it (probably by doing this #2267 (comment))

]
},
"rebuild": {
"onlyModules": ["interruptor", "keytar", "kerberos"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I guess eventually we want mongodb-client-encryption to be a part of this list so it can be used in Compass shell, but for now this doesn't change any current behavior

@gribnoysup gribnoysup merged commit 731ad40 into master Jun 21, 2021
@gribnoysup gribnoysup deleted the third-time-is-the-charm branch June 21, 2021 08:22
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