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

JBIDE-18277: Update earlyaccess listing #296

Merged
merged 1 commit into from
Sep 29, 2014
Merged

JBIDE-18277: Update earlyaccess listing #296

merged 1 commit into from
Sep 29, 2014

Conversation

mickaelistria
Copy link
Contributor

No description provided.

@fbricon
Copy link
Member

fbricon commented Sep 29, 2014

Why is this even stored in git, since it's updated via mvn on each build?

@mickaelistria
Copy link
Contributor Author

That's a good point. I could remove the file if you prefer.

@fbricon
Copy link
Member

fbricon commented Sep 29, 2014

Yup, I think removing the file + updating .gitignore will require less maintenance

@mickaelistria
Copy link
Contributor Author

Actually, this file is currently not in repo. I thought it was in and required an update, but since it's not it and it's already fetched at each build, there is nothing to update here.
The ,gitignore could list the file, but it's not worth it for now.

@fbricon
Copy link
Member

fbricon commented Sep 29, 2014

if it's not in the repo, how does git see it as modified then? :-)

@mickaelistria mickaelistria reopened this Sep 29, 2014
Listing is fetched at build time
@mickaelistria
Copy link
Contributor Author

Ok, I pushed a commit removing the file and adding it to ,gitignore.

@mickaelistria
Copy link
Contributor Author

@fbricon : If you're ok with it, please merge it to jbosstools-4.2.x and cherry-pick it to master.

@fbricon fbricon merged commit 4a3d710 into jbosstools:jbosstools-4.2.x Sep 29, 2014
@fbricon
Copy link
Member

fbricon commented Sep 29, 2014

applied on both branches

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.

None yet

2 participants