-
Notifications
You must be signed in to change notification settings - Fork 33
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
Fix for issue #185 and #187 #188
Conversation
… despite "dotnet new --list" showing some.
@scheind |
Can you please sign the Eclipse contributor agreement https://accounts.eclipse.org/user/login?destination=legal/eca/validation/129563 ? |
So I created a account at eclipse and signed the ECA but it still says that it could not validate the sign-off status. Do I have to do something additionally? |
Could you please check the email address is matched with the email you register for Eclipse? |
@eriawan Yes seems to be the same |
@scheind |
I do not see any obvious reason why this would fail; the email address mismatch between the commit author and the Eclipse account is indeed the usual suspect, but if this is not the cause here, I don't know what else to suggest. Maybe @waynebeaton can tell more... |
Latest fedora, java, docker, etc.
Would you also please rebase on top of master where build should be fixed and your changes should hopefully fix the failing tests. |
I've tried this patch on top of master with dotnet 6 and new wizard still doesn't have templates. |
And remove java nature, classpath, etc. where not needed instead of keeping it at 1.8.
Whatever was the problem, it appears to have been fixed (note that if you follow the "Details" link, you can rerun the validation). As a matter of practice, open an issue against the Eclipse Foundation help desk when you encounter these sorts of problems. The IT team are the ones who maintain this infrastructure and we really need them to diagnose these sorts of issues if we have any hope of improving it. |
… despite "dotnet new --list" showing some.
@akurtakov Was this in the tests? I synced the repository and tried it again on my end. I used dotnet 6 and 7 (although not really different) and the templates where displayed. |
no, I cherry-picked your commit on master locally and tried it. I'll have to look further. |
Sorry for the many comments. This is really my first time with git rebase so I'm not to sure. What exactly do I have to do? |
Assuming you have this repo as "upstream" remote (added via
|
Not familiar with the tool used . Just push it :) |
ok pushed it and it says "everything up-to-date" |
@scheind could you fix the test failures? |
@eriawan The test failures in Jenkins seem to be related to another issue since it couldn't find the menu bar for the shell. But I don't know enough about SWTBotTest or how it works. |
@scheind Currently next week I'm going to focus on my own daily works on some on .NET Core, including some works on NET 7.0 too. Then have a long vacation starting from mid December. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for now!
To other collaborators, feel free to chime in. 🙂 |
Yes no problem, despite being a little late (3 months or so 😄 ) |
Sorry for not creating the PR sooner. Kinda lost track of it