-
Notifications
You must be signed in to change notification settings - Fork 655
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
Feature project json for discussion (By Droyad) #1094
Conversation
…on in the original file.
…at is a VisualStudioism.
FORGET IT HAHA :( |
@dazinator what happened? |
The failed checks with Travis and AppVeyor are going to take too much time for me to investigate right now. I had a quick glimpse at the first one, and it looked like AppVeyor is complaining about a missing release notes md file, and Travis actually gets as far as running the tests but for some reason the tests fail. I was hoping it might just be a simple merge but it looks like its not going to be. I'm also not fully familiar with AppVeypr or Travis yet so I am not the best person to look at this until I can get up to speed. The original PR is still open #980 |
That is a known issue that I have an open PR to correct. Likely not an issue with the work you have done.
This is likely nothing to do with you either. Have a look at the conversation here: #1093 (comment) |
Ah ok nice one, perhaps i'll revisit this again after then, if necessary.
Disclaimer: none of this is my work - this is all @droyad 's - i just thought I'd have a quick go at trying to get the checks passing so it could be integrated. Will accept defeat for now. |
@dazinator: Thanks for giving this a go! Unfortunately, GitVersion (or rather, its CI pipeline) isn't in very good shape at the moment. I hope we have that fixed soon and that you can take another stab at rebasing this once the CI pipeline has been stabilised. |
Should be good to try again now :) |
I have been using @droyad 's branch with the project json feature, and it has proved very useful.
Just gonna have a go at getting checks passed so it can be merged.