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.

Sign up for GitHub

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

Jump to bottom

Clean up error messages to have consistent format #235

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Clean up error messages to have consistent format #235

merged 2 commits into from
Sep 25, 2024

Conversation

joshuahannan
Copy link
Member

@joshuahannan joshuahannan commented Sep 25, 2024 •
edited
Loading

Description

Removes all instances of = in error messages and do other small fixes so they have a more consistent format

Based on guidelines in onflow/docs#795

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Re-reviewed Files changed in the Github PR explorer

sisyphusSmiling
sisyphusSmiling approved these changes Sep 25, 2024
View reviewed changes
contracts/utility/NFTForwarding.cdc Outdated Show resolved Hide resolved
@joshuahannan joshuahannan merged commit d3bdc87 into master Sep 25, 2024
2 checks passed
@joshuahannan joshuahannan deleted the small-fixes branch September 25, 2024 22:06
@ksvnar
Copy link

ksvnar commented Sep 28, 2024

Спасибо вам

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sisyphusSmiling sisyphusSmiling sisyphusSmiling approved these changes

@nvdtf nvdtf Awaiting requested review from nvdtf

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joshuahannan @ksvnar @sisyphusSmiling