Skip to content

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 34 commits into from
Mar 28, 2025
Merged

Update to Gleam 1.8.1 #53

merged 34 commits into from
Mar 28, 2025

Conversation

PgBiel
Copy link
Member

@PgBiel PgBiel commented Mar 28, 2025

Closes #52

PgBiel added 30 commits March 28, 2025 03:17
see df53d2f
included with 555c25c (Gleam 1.4.0
update)
fix clippy is_none_or

temporary workaround for project_path_deps test
Please use 'bin/glistix-fix-test-names.sh' to auto-generate this commit
instead of frustratingly attempting to resolve conflicts.
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
cf f64f24f (gh:#51)

* add deprecation warning

* use exclamation marks
recommend depending on glistix_stdlib by default: see 2a26306

add book links: see f0f0df3

change 1: see aeb3b9f
(gh:#44) and 408dfdf (gh:#48)

initial: see 64710af,
c7b59c2,
a308328,
1b08191 and related commits
@PgBiel PgBiel added this to the v0.8.0 milestone Mar 28, 2025
@PgBiel PgBiel force-pushed the gleam-bump/1.8.1 branch from 1006546 to 9a5a77c Compare March 28, 2025 06:33
@PgBiel PgBiel merged commit 8dfaf6c into main Mar 28, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bump to Gleam 1.8.1
1 participant