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

Explicitly use the class name that is being deprecated #4838

Closed
wants to merge 3 commits into from

Conversation

elia
Copy link
Member

@elia elia commented Jan 5, 2023

Summary

The message is otherwise hard to understand.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

  • I have written a thorough PR description.
  • I have kept my commits small and atomic.
  • I have used clear, explanatory commit messages.

The following are not always needed (cross them out if they are not):

- [ ] I have added automated tests to cover my changes. - [ ] I have attached screenshots to demo visual changes. - [ ] I have opened a PR to update the [guides](https://github.com/solidusio/edgeguides). - [ ] I have updated the README to account for my changes.

@elia elia requested a review from a team as a code owner January 5, 2023 22:44
@elia elia self-assigned this Jan 5, 2023
@github-actions github-actions bot added the changelog:solidus_core Changes to the solidus_core gem label Jan 5, 2023
@elia elia mentioned this pull request Jan 5, 2023
3 tasks
@codecov
Copy link

codecov bot commented Jan 5, 2023

Codecov Report

Merging #4838 (accab6d) into master (4451065) will not change coverage.
The diff coverage is 100.00%.

❗ Current head accab6d differs from pull request most recent head f4f4086. Consider uploading reports for the commit f4f4086 to get more accurate results

@@           Coverage Diff           @@
##           master    #4838   +/-   ##
=======================================
  Coverage   86.16%   86.16%           
=======================================
  Files         578      578           
  Lines       14664    14664           
=======================================
  Hits        12635    12635           
  Misses       2029     2029           
Impacted Files Coverage Δ
core/app/models/spree/user_last_url_storer.rb 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🙏

The message is otherwise hard to understand.
@elia elia force-pushed the elia/improve-deprecation-message branch from 68acd75 to accab6d Compare January 9, 2023 09:16
@waiting-for-dev waiting-for-dev added the type:enhancement Proposed or newly added feature label Jan 16, 2023
@kennyadsl
Copy link
Member

@elia should we close this PR now that #4856 has been merged?

@elia elia closed this Jan 18, 2023
@kennyadsl kennyadsl deleted the elia/improve-deprecation-message branch May 8, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_core Changes to the solidus_core gem type:enhancement Proposed or newly added feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants