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.

Sign up for GitHub

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

Jump to bottom

hdevtools check src/Main.hs errors on its own source #23

Open
schell opened this issue Oct 23, 2015 · 7 comments · May be fixed by #24
Open

hdevtools check src/Main.hs errors on its own source #23

schell opened this issue Oct 23, 2015 · 7 comments · May be fixed by #24

Comments

@schell
Copy link
Owner

schell commented Oct 23, 2015

After merging the last set of PRs (#18,#19,#20,#21,#22) hdevtools errors when checking its own source. It seems to be fine when check is run on projects that are less complex. Maybe this has to do with the stack updates?
screenshot 2015-10-23 14 26 59
@ch1bo
Copy link

ch1bo commented Oct 24, 2015

I had the same issue
2015-10-24-041439_815x260_scrot

Seems that the change f71e9b2 always tries to read the dist folder and fails when it does not exist (e.g. when only built via stack). I'll open a PR from my fork.

@ch1bo ch1bo linked a pull request Oct 24, 2015 that will close this issue
Open
@ch1bo ch1bo mentioned this issue Nov 23, 2015
Merged
@ch1bo
Copy link

ch1bo commented Jan 19, 2016

As this might still have some visiblity can we resolve the issue and/or forward people to the new hdevtools/hdevtools repository?

@schell
Copy link
Owner Author

schell commented Jan 19, 2016

Yes - how should I forward people there?

@ch1bo
Copy link

ch1bo commented Jan 19, 2016

Maybe in the project title / README.md?

@schell
Copy link
Owner Author

schell commented Jan 19, 2016

@ch1bo - done! Can you guys release a new version of hdevtools on hackage with the new repository details?

@ch1bo
Copy link

ch1bo commented Jan 19, 2016

I just did a couple days/weeks ago. Current version 0.1.2.2 is also in the Stackage lts-4.2.

@schell
Copy link
Owner Author

schell commented Jan 19, 2016

Oh! Sorry! News to me ;) You guys rock. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check if 'dist' exists when looking for snapshot dist
2 participants
@schell @ch1bo