-
-
Notifications
You must be signed in to change notification settings - Fork 2
Update to Gleam 1.8.1 #53
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix clippy is_none_or temporary workaround for project_path_deps test
see c3fb3c3 for original commit
Please use 'bin/glistix-fix-test-names.sh' to auto-generate this commit instead of frustratingly attempting to resolve conflicts.
see 17225bc and others
replaces dependencies when exporting (temporary) see 83a630a
Ensure root project's local dependencies take precedence over transitive local dependencies of same name see cdf2857 and subsequent commits
moved config locked changes to stale_package_remover cf gh:#44 cf f0f0df3 (add more patch info to errors) cf 408dfdf (additional patching fixes - gh:#48) * patching attempt 2 - beginnings * rename replacements to glistix_replacements for clarity help when merging stuff in the future * patch manifest requirements needed for the locked() function * fix inconsistency between manifest and compiled package config During compilation, dependencies' configs must be updated, not only in the CLI side when generating the manifest or reading the root config. Otherwise, we'd get a warning saying that the dependency is importing a module from something that it doesn't depend on, which it can only suppose to be a transitive dependency. * default config read to patching * rename replacements to patches * test glistix patches in dependency resolving * make patch rename optional * glistix patch: allow not specifying name, flatten source * send unpatched hex dependencies when publishing Ideally docs would be unpatched as well, but then we can't build. * add patches to manifest needed to know if they changed * always unlock packages affected by new patches For example, if we patch gleam_stdlib and rename it to glistix_stdlib, now any packages depending on gleam_stdlib OR glistix_stdlib, as well as both packages themselves (if they appear in the manifest), should be considered stale and removed from the manifest. * consider removed patches when unlocking * use patch in 'glistix new' Back out "cli/new: automatically clone stdlib,gleeunit" This backs out commit c7b59c2. * clippy and update tests * fix windows tests * don't make patched hex dependency a root dependency unless necessary * add glistix_project_nix test * add error hints suggesting patches * improve link display in error message * revert some clippy changes * prompt version resolving when patches differ * further changes to default gleam.toml - remove hex-patch suggestion * add test/glistix_project_nix to CI * improve renamedpkg example in gleam.toml * add 'maybe forgot patch hint' to unsupported nix target feature * check for 'build/packages' for maybe forgot patch error hint * add a comment to manifest requirement renaming * regen manifest after 'glistix add' if patched dep was added * remove unused patching functions * add todo for better unlocking behavior in the future * fix glistix new giving wrong version override suggestion * add config patch tests * fix: also patch dev deps * fix: check for patch conflict, deterministic patch order * small spacing changes in config.rs * fix patching with local deps using relative paths * add local patching test * add glistix_hex_patching test * fix unnecessary clone and add todo * discourage local-overrides setting Additional patching fixes (#48): * fix language server patch * default to gleam_stdlib dep by default I think this might make more sense, might revert it later
1006546
to
9a5a77c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #52