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

fix: use only one version of ic-certification #1267

Merged
merged 2 commits into from
Sep 2, 2024

Conversation

ninegua
Copy link
Member

@ninegua ninegua commented Sep 2, 2024

The external ic-certification package was used by multiple dependencies, but with slightly different versions. This PR makes sure only one version is used. To help solve the problem, the ic-http-proxy dependency is also pinned to a more recent version.

Note that the external ic-certification is a different package than the one internal in rs/certification subdir.

@github-actions github-actions bot added the fix label Sep 2, 2024
@ninegua ninegua marked this pull request as ready for review September 2, 2024 13:10
@ninegua ninegua requested a review from a team as a code owner September 2, 2024 13:10
@github-actions github-actions bot added the @idx label Sep 2, 2024
@ninegua ninegua added this pull request to the merge queue Sep 2, 2024
Merged via the queue into master with commit 6760029 Sep 2, 2024
29 checks passed
@ninegua ninegua deleted the paulliu/unify-ic-certifcation-versions branch September 2, 2024 14:11
levifeldman pushed a commit to levifeldman/ic that referenced this pull request Oct 1, 2024
The external ic-certification package was used by multiple dependencies,
but with slightly different versions. This PR makes sure only one
version is used. To help solve the problem, the ic-http-proxy dependency
is also pinned to a more recent version.

Note that the external ic-certification is a different package than the
one internal in rs/certification subdir.

---------

Co-authored-by: IDX GitHub Automation <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants