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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃彈Ignore react-dates related packages #21126

Merged
merged 3 commits into from Feb 27, 2019
Merged

Conversation

cvializ
Copy link
Contributor

@cvializ cvializ commented Feb 26, 2019

Related to #21092

Should permanently resolve issues with renovate upgrading react-dates when we don't actually want to upgrade it.

Copy link
Contributor

@rsimha rsimha left a comment

Choose a reason for hiding this comment

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

I see an empty src/test.js. Unintentional?

"moment-with-locales",
"react-addons-shallow-compare",
"react-externs",
"react-with-direction",
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Alphabetize this list for easy maintenance.

Copy link
Contributor

@rsimha rsimha left a comment

Choose a reason for hiding this comment

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

In description: s/greenkeeper/renovate/ :)

@cvializ
Copy link
Contributor Author

cvializ commented Feb 27, 2019

Thanks for the review!

@cvializ cvializ merged commit d0bf3c9 into ampproject:master Feb 27, 2019
noranazmy pushed a commit to noranazmy/amphtml that referenced this pull request Mar 22, 2019
* Ignore react-dates related packages

* Sort and cleanup

* Update yarn.lock
bramanudom pushed a commit to bramanudom/amphtml that referenced this pull request Mar 22, 2019
* Ignore react-dates related packages

* Sort and cleanup

* Update yarn.lock
@cvializ cvializ deleted the fix/deps branch April 2, 2019 20:58
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.

None yet

3 participants