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

Create Frontend package #216

Merged
merged 1 commit into from
Dec 1, 2021
Merged

Create Frontend package #216

merged 1 commit into from
Dec 1, 2021

Conversation

knothed
Copy link
Contributor

@knothed knothed commented Nov 24, 2021

This PR is pretty straightforward, it moves all frontend-specific files into the happy-frontend package.

@knothed
Copy link
Contributor Author

knothed commented Nov 24, 2021

@Ericson2314 @int-index

@knothed
Copy link
Contributor Author

knothed commented Nov 24, 2021

Todo: adapt Haskell-CI (note to myself)

@knothed knothed force-pushed the frontend branch 2 times, most recently from 11bc4e3 to 877b12d Compare November 26, 2021 15:11
@knothed
Copy link
Contributor Author

knothed commented Nov 26, 2021

Todo: adapt Haskell-CI (note to myself)

Done. Seems to work.

Copy link
Collaborator

@Ericson2314 Ericson2314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This looks good to me now. @int-index?

@int-index
Copy link
Collaborator

I haven’t looked at this as carefully as at the previous patches, but overall this looks good to me.

@Ericson2314
Copy link
Collaborator

This diff is indeed simpler to read, so I don't think there is anything we missed. I'll merge it. Thanks again, @knothed!

@Ericson2314 Ericson2314 merged commit 11e3e78 into haskell:master Dec 1, 2021
@Ericson2314 Ericson2314 deleted the frontend branch December 1, 2021 16:16
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

3 participants