Skip to content

chore(nightly): update proc-macro span file name method name #3852

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 1 commit into from
Apr 16, 2025

Conversation

gbj
Copy link
Collaborator

@gbj gbj commented Apr 16, 2025

Latest nightly 2025-04-15 removes the .source_file() method on proc_macro::Span (used in our hot-reloading code) and splits it into .file() and local_file().

The bad news: this means leptos nightly no longer compiles on latest nightly.

The good news: this is a step toward stabilization for other parts of the API, which is great.

I've made the decision to just change to the newer method name, and require nightly users to use latest nightly.

@benwis
Copy link
Contributor

benwis commented Apr 16, 2025

Thanks @gbj, seems reasonable for the nightly folks

@benwis benwis merged commit 8f38559 into main Apr 16, 2025
263 checks passed
killshot-karma pushed a commit to killshot-karma/leptos that referenced this pull request Apr 18, 2025
NKID00 pushed a commit to NKID00/leptos that referenced this pull request Apr 24, 2025
sabify added a commit to sabify/leptos that referenced this pull request May 2, 2025
sabify added a commit to sabify/leptos that referenced this pull request May 2, 2025
gbj added a commit that referenced this pull request May 4, 2025
* fix(CI): prevent regreession from nightly clippy in autofix

* chore: format

* chore: update nightly to 2025-04-16 (proc-macro span, #3852)

* chore: improve the autofix ci workflow

* fix: adjust ServerFn macro test stderr based on nightly-2025-04-16

* fix: limit server_fn server macro trybuild tests nightly only
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