Skip to content

Rename new_user_event to with_user_event#1068

Merged
goddessfreya merged 1 commit intorust-windowing:masterfrom
briankabiro:update-method-name
Jul 29, 2019
Merged

Rename new_user_event to with_user_event#1068
goddessfreya merged 1 commit intorust-windowing:masterfrom
briankabiro:update-method-name

Conversation

@briankabiro
Copy link
Copy Markdown
Contributor

Finishes #1057

  • Tested on all platforms changed
  • cargo fmt has been run on this branch
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality
  • Updated feature matrix, if new features were added or implemented

@briankabiro briankabiro changed the title Rename new_user_event to new_with_user_event (#1057) Rename new_user_event to new_with_user_event Jul 24, 2019
@goddessfreya
Copy link
Copy Markdown
Contributor

#1057 (comment)

@briankabiro
Copy link
Copy Markdown
Contributor Author

I think I had misinterpreted the comment but updated the method name correctly now.

@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 25, 2019

Thanks for implementing this! All we need is a changelog entry and this should be good to merge.

@briankabiro briankabiro changed the title Rename new_user_event to new_with_user_event Rename new_user_event to with_user_event Jul 29, 2019
@briankabiro
Copy link
Copy Markdown
Contributor Author

All done updating the CHANGELOG.

@goddessfreya
Copy link
Copy Markdown
Contributor

Can you try re-pushing? Appveyor seams angry.

@briankabiro
Copy link
Copy Markdown
Contributor Author

Looks better now. 😃

@goddessfreya goddessfreya merged commit 131e67d into rust-windowing:master Jul 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants