Skip to content

Webmerge3#1037

Merged
goddessfreya merged 53 commits into
rust-windowing:webfrom
goddessfreya:webmerge3
Jul 10, 2019
Merged

Webmerge3#1037
goddessfreya merged 53 commits into
rust-windowing:webfrom
goddessfreya:webmerge3

Conversation

@goddessfreya
Copy link
Copy Markdown
Contributor

  • 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

elliegoldstein and others added 30 commits February 12, 2019 20:47
Everything typechecks, but nothing is implemented
Update the internal APIs to match the new API changes
@goddessfreya goddessfreya merged commit 5696357 into rust-windowing:web Jul 10, 2019
goddessfreya added a commit that referenced this pull request Jul 10, 2019
Actually compile please.

Signed-off-by: Hal Gentz <zegentzy@protonmail.com>
@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 10, 2019

Hold up, did we get approval from the parties involved to merge this ourselves? I didn't get any notifications about GitHub discussion on this. cc @hecrj @blm768 @ryanisaacg

Generally speaking, I'm not comfortable having people submit PRs on other people's behalf, since that fucks up the git attributions.

@goddessfreya
Copy link
Copy Markdown
Contributor Author

@Osspial This was done with @ryanisaacg's approval. We discussed this on Discord. Apologies for not cluing you in.

@goddessfreya goddessfreya deleted the webmerge3 branch July 10, 2019 05:19
@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 10, 2019

@zegentzy Discussing things off of GitHub is fine, it's just that this was a big merge done rather suddenly. I'm okay with doing this, but I'd at least like to have each author's work squashed separately, so that it's clear who did what when going through the git history.

@goddessfreya
Copy link
Copy Markdown
Contributor Author

@Osspial Squashing changes by author will take millennia, and each author's contributions probably will not compile without the final merge. I'd rather we either squash them all, or not squash them at all.

@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 10, 2019

@zegentzy could you re-push your webmerge3 branch? I can see if there's a way to squash per-author easily. welp you've already got a branch

@goddessfreya
Copy link
Copy Markdown
Contributor Author

goddessfreya commented Jul 10, 2019

#1038 (webmerge3 had some errors)

@felixrabe
Copy link
Copy Markdown
Contributor

Re squashing: Please don't. Or at least keep the original commits around in a "web-orig" branch that is identical to the squashed version.

@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 10, 2019

I've enabled non-squashing merges, which preserves the author history.

@blm768
Copy link
Copy Markdown
Contributor

blm768 commented Jul 11, 2019

FWIW, my changes have probably been largely overwritten in the process of @hecrj's refactoring, so I'm not sure it's worth making too much effort to preserve those particular changes' history. I'm not terribly worried about attribution; I'm just excited that I'll finally get to use winit on the Web.

@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jul 12, 2019

@blm768 I can get not being too worried about attribution from a user's perspective, but doing it wrong is something we could get in trouble for and has the potential to blow up into a debacle so I'm keen on doing it right if at all possible.

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.

6 participants