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: Update dependencies #55

Merged
merged 6 commits into from
Jan 7, 2025
Merged

chore: Update dependencies #55

merged 6 commits into from
Jan 7, 2025

Conversation

Gashmob
Copy link
Member

@Gashmob Gashmob commented Dec 27, 2024

Closes #54

Part of #54

Commit 3e889463e499537c602e3ea83da6e33f9dc974da

+ create pnpm derivation directly from version 9.15.1
@Gashmob Gashmob added the dependencies Update/Change dependencies label Dec 27, 2024
@Gashmob Gashmob added this to the v0.1.0 milestone Dec 27, 2024
@Gashmob Gashmob self-assigned this Dec 27, 2024
@Gashmob Gashmob linked an issue Dec 27, 2024 that may be closed by this pull request
Part of #54

`tools/bin` is added to $PATH when in nix-shell, so you can invoke the
script directly. Paths inside scripts are also now absolute and use var
$ROOT_DIR defined in nix-shell. So you need to be in nix-shell in order
to use scripts, else it may cause some troubles.
@Gashmob Gashmob force-pushed the 54-update-dependencies branch 2 times, most recently from fd86081 to 906b1cf Compare December 29, 2024 15:13
Part of #54

Instead of using FetchContent in cmake to download libraries, use
nix-shell to add them as dependencies. Cmake can then retrieve them.

When in nix-shell, nix store paths are already added to cmake vars. But
when using an IDE not aware of nix environment, we need to add it
manually -> line 4 to 9 in CMakeLists.txt are here for that
@Gashmob Gashmob force-pushed the 54-update-dependencies branch from 906b1cf to a83f8e7 Compare December 29, 2024 15:13
Part of #54

After the move of scripts in 9bbd82c,
package and tests workflow were updated in order to use new location but
not codeql. This commit fix this miss
Part of #54

With moving of dependencies, some ignores of lcov are not matched and
considered as errors

Also choose to ignore version errors
Part of #54

3e889463e499537c602e3ea83da6e33f9dc974da has a broken rpm/rpmbuild

eb28b94bd14835836b539bc3854a6abf929876d4 is previously used pin and it
works with it
@Gashmob Gashmob merged commit 00baaa0 into master Jan 7, 2025
13 checks passed
@Gashmob Gashmob deleted the 54-update-dependencies branch January 7, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Update/Change dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update dependencies
1 participant