Skip to content

Conversation

LFDanLu
Copy link
Member

@LFDanLu LFDanLu commented Dec 4, 2020

Closes

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@adobe-bot
Copy link

Build successful! 🎉

@adobe-bot
Copy link

Build successful! 🎉

snowystinger
snowystinger previously approved these changes Dec 5, 2020
@devongovett
Copy link
Member

If it's an alpha/beta/rc, it should not use a caret range, it should be fixed. This is because we allow breaking changes but technically semver sees later prereleases as compatible.

@LFDanLu
Copy link
Member Author

LFDanLu commented Dec 7, 2020

@devongovett Ah, thanks for the explanation, will fix so it detects the alpha/beta/rc when deciding whether or not to add a caret.

@adobe-bot
Copy link

Build successful! 🎉

snowystinger
snowystinger previously approved these changes Dec 8, 2020
@dannify
Copy link
Member

dannify commented Dec 23, 2020

Sorry for the conflicts

@adobe-bot
Copy link

Build successful! 🎉

Copy link
Member

@ktabors ktabors left a comment

Choose a reason for hiding this comment

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

LGTM

@adobe-bot
Copy link

Build successful! 🎉

@LFDanLu LFDanLu merged commit 7326ad2 into main Jan 19, 2021
@LFDanLu LFDanLu deleted the fixing_lint_import branch January 19, 2021 19:22
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.

6 participants