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

Makefiles make things easier for new and regular contributors alike #3266

Merged
merged 3 commits into from
Mar 30, 2018
Merged

Makefiles make things easier for new and regular contributors alike #3266

merged 3 commits into from
Mar 30, 2018

Conversation

bitemyapp
Copy link
Contributor

I use a Makefile in almost all of my software projects as a sort of exobrain/secondary memory and regularized shortcut mechanism for doing common tasks. PureScript didn't have one, so I've PR'd one based on my small but burgeoning collection at https://github.com/bitemyapp/makefiles from which https://github.com/bitemyapp/lefortovo grabs things.

I also updated the .gitignore to ignore profiling bumf.

@kritzcreek
Copy link
Member

I probably won't use this much as I don't have make installed on Windows, but having a place to collect a few commonly run commands doesn't sound like a bad idea, and if it's executable for most that's a plus, so 👍 from me.

@bitemyapp
Copy link
Contributor Author

@kritzcreek does this need anything else to be merged?

@parsonsmatt
Copy link
Contributor

I forgot that this even existed when I made my own makefile PR 😮

@bitemyapp mind if I PR this branch with my ghcid, ghcid-test, and help commands?

@bitemyapp
Copy link
Contributor Author

@parsonsmatt go ahead!

@parsonsmatt
Copy link
Contributor

I failed to use GitHub correctly and now my PR to your PR is borked, so I'll just update my PR after this one lands.

Unless PS folks want to do it all in one go, at which point I will retry.

@garyb
Copy link
Member

garyb commented Mar 30, 2018

I don't mind if we do it in one or add it afterwards. This looks good to me too! @kritzcreek got any comments?

@kritzcreek kritzcreek merged commit 80cef84 into purescript:master Mar 30, 2018
@bitemyapp bitemyapp deleted the bitemyapp/springtime-is-make-time branch March 31, 2018 04:09
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.

4 participants