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

Run astyle script on all source files #5

Open
giorgiobornia opened this issue Feb 17, 2015 · 2 comments
Open

Run astyle script on all source files #5

giorgiobornia opened this issue Feb 17, 2015 · 2 comments

Comments

@giorgiobornia
Copy link
Contributor

No description provided.

giorgiobornia added a commit that referenced this issue Mar 7, 2015
Merge some XDMF changes
giorgiobornia pushed a commit that referenced this issue Apr 9, 2015
Merge from main repo hex27
@giorgiobornia
Copy link
Contributor Author

Some time during this summer we will run the astyle script on all source files.
Plus, for the future, I think we should define a test to be run by jenkins every time the master is pushed to. The test should do the following every time:
run the astyle script on all source files;
if no formatting took place, then leave everything as it is;
if some formatting took place, then do an additional commit that fixes the formatting (pushing it back to github...).

I have to think about that.

The other non-automatic alternative is to distribute the script to all developers and tell them where to put it in their local git repo (since the hooks are not tracked).

@eaulisa
Copy link
Contributor

eaulisa commented May 23, 2015

whenever you want to do it I am Okay
my fork is synchronized with the master branch
By the way now we also have MG from PETSC

@eaulisa eaulisa closed this as completed May 23, 2015
@eaulisa eaulisa reopened this May 23, 2015
giorgiobornia pushed a commit that referenced this issue Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants