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

Feature project json for discussion (By Droyad) #1094

Closed
wants to merge 8 commits into from

Conversation

dazinator
Copy link
Member

@dazinator dazinator commented Nov 20, 2016

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.

@dazinator
Copy link
Member Author

dazinator commented Nov 20, 2016

FORGET IT HAHA :(

@dazinator dazinator closed this Nov 20, 2016
@gep13
Copy link
Member

gep13 commented Nov 20, 2016

@dazinator what happened?

@dazinator
Copy link
Member Author

dazinator commented Nov 20, 2016

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

@gep13
Copy link
Member

gep13 commented Nov 20, 2016

@dazinator said...
and it looked like AppVeyor is complaining about a missing release notes md file,

That is a known issue that I have an open PR to correct. Likely not an issue with the work you have done.

@dazinator said...
running the tests but for some reason the tests fail

This is likely nothing to do with you either. Have a look at the conversation here: #1093 (comment)

@dazinator
Copy link
Member Author

dazinator commented Nov 20, 2016

That is a known issue that I have an open PR to correct. Likely not an issue with the work you have done.

Ah ok nice one, perhaps i'll revisit this again after then, if necessary.

Likely not an issue with the work you have done.

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.

@asbjornu
Copy link
Member

@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.

@JakeGinnivan
Copy link
Contributor

Should be good to try again now :)

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

Successfully merging this pull request may close these issues.

5 participants