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

deprecate attribute: renamed reason to note #1425

Closed
wants to merge 26 commits into from

Conversation

llogiq
Copy link
Contributor

@llogiq llogiq commented Dec 23, 2015

This change was discussed (and accepted) on #1270 before the RFC PR was merged; unfortunately, I forgot to change the RFC text. The current implementation has the field named as "note" and is thus in violation of the RFC as merged. This PR should correct this oversight.

llogiq added 26 commits June 3, 2015 15:56
@ticki
Copy link
Contributor

ticki commented Dec 23, 2015

👍

@brson
Copy link
Contributor

brson commented Dec 23, 2015

This history looks very suspicious.

@llogiq
Copy link
Contributor Author

llogiq commented Dec 23, 2015

Yeah, it based on an old commit. I'm not sure whether to rebase or just close and resubmit with a clean branch?

@ticki
Copy link
Contributor

ticki commented Dec 23, 2015

@llogiq rebase is being a jerk, whenever you have already pushed, so I guess the best thing to do is to open a new PR.

@llogiq
Copy link
Contributor Author

llogiq commented Dec 23, 2015

OK then.

@llogiq llogiq closed this Dec 23, 2015
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.

3 participants