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

Replace DEPRECATED with @deprecated #1054

Merged
merged 3 commits into from
Jun 23, 2022
Merged

Conversation

chmanie
Copy link
Member

@chmanie chmanie commented Jun 20, 2022

Closes #1035

I also took the liberty to fix some of the NatSpec warnings we got when running build:docs.

@chmanie chmanie force-pushed the maint/replace-deprecated-natspec-tag branch from a2436e2 to dd974d5 Compare June 20, 2022 19:10
@chmanie
Copy link
Member Author

chmanie commented Jun 20, 2022

Ah, I see, okay. @deprecated is not a natspec tag. Yeah, fine. I'll work around that.

@area
Copy link
Member

area commented Jun 21, 2022

What does that mean for this PR?

@chmanie
Copy link
Member Author

chmanie commented Jun 21, 2022

That we likely have to use

/// @notice @deprecated

instead. Sorry I didn't get to adjusting that.

@chmanie chmanie force-pushed the maint/replace-deprecated-natspec-tag branch from 6c710f3 to c545442 Compare June 21, 2022 16:17
@chmanie chmanie marked this pull request as ready for review June 21, 2022 16:36
@area
Copy link
Member

area commented Jun 22, 2022

So the issue here is that you've changed VotingReputation. There are no functional changes, but the script isn't bright enough to realise that and thinks it should have its version bumped.

I'm not really sure what to do about this, I'm going to be fairly resistant to changing that script.

@chmanie chmanie force-pushed the maint/replace-deprecated-natspec-tag branch from c545442 to e8e4060 Compare June 22, 2022 21:06
@area area force-pushed the maint/replace-deprecated-natspec-tag branch from e8e4060 to 7eb39f2 Compare June 23, 2022 10:25
@area area merged commit 9afd5ce into develop Jun 23, 2022
@area area deleted the maint/replace-deprecated-natspec-tag branch June 23, 2022 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace DEPRECATED with @deprecated in the dev docs
2 participants