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

Simplify TopLevel and Static export handling #4170

Merged
merged 4 commits into from Sep 6, 2020
Merged

Conversation

@gzm0
Copy link
Contributor

gzm0 commented Aug 30, 2020

No description provided.

@gzm0 gzm0 force-pushed the gzm0:cleanup-exports branch 2 times, most recently from 9647182 to ffd3e76 Aug 30, 2020
@gzm0 gzm0 marked this pull request as ready for review Aug 30, 2020
@gzm0 gzm0 requested a review from sjrd Aug 30, 2020
@gzm0 gzm0 force-pushed the gzm0:cleanup-exports branch from ffd3e76 to 283a94a Aug 30, 2020
Copy link
Member

sjrd left a comment

Looks good. I only have minor comments.

gzm0 added 2 commits Aug 30, 2020
The statement that sym.isMethod is true in general is wrong: adding an
assert makes the compiler crash for nested classes / objects.
@gzm0 gzm0 force-pushed the gzm0:cleanup-exports branch from 283a94a to 7974240 Sep 6, 2020
@gzm0 gzm0 requested a review from sjrd Sep 6, 2020
@gzm0
Copy link
Contributor Author

gzm0 commented Sep 6, 2020

The AppVeyor build seems to have suffered an infrastructure failure. Is there a way I can restart it?

@sjrd
Copy link
Member

sjrd commented Sep 6, 2020

The AppVeyor build seems to have suffered an infrastructure failure. Is there a way I can restart it?

If you login into AppVeryor with your GitHub account, you should be able to trigger re-builds. I've done it for this build now.

@sjrd
sjrd approved these changes Sep 6, 2020
@gzm0
Copy link
Contributor Author

gzm0 commented Sep 6, 2020

If you login into AppVeryor with your GitHub account, you should be able to trigger re-builds. I've done it for this build now.

I can't find the button. Are you sure I have the appropriate permissions? If I click on "Projects" in the nav bar, the scala-js repository does not show up.

@sjrd
Copy link
Member

sjrd commented Sep 6, 2020

You're an Owner of the scala-js organization, so you should have all privileges. Perhaps you need to explicitly sync your AppVeyor profile with your GitHub permissions?

@gzm0
Copy link
Contributor Author

gzm0 commented Sep 6, 2020

Perhaps you need to explicitly sync your AppVeyor profile with your GitHub permissions?

How do I do that? I have tried: Installing the AppVeyor GitHub app for myself (gzm0) and the scala-js organization.

@sjrd sjrd merged commit 8cbb316 into scala-js:master Sep 6, 2020
3 checks passed
3 checks passed
check
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
@gzm0 gzm0 deleted the gzm0:cleanup-exports branch Sep 6, 2020
@sjrd
Copy link
Member

sjrd commented Sep 6, 2020

Apparently the AppVeyor builds are configured with my personal account, instead of through the scala-js org. The problem is that I have no idea how to fix it, nor even to recreate the project under the scala-js org from the AppVeyor point of view :s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.