Skip to content

Conversation

@stevladimir
Copy link
Contributor

@stevladimir stevladimir commented Apr 18, 2025

  • Added cabal.project
  • Added CI
  • Extended set of warnings and default-extensions
  • Added .gitignore

Copy link
Member

Choose a reason for hiding this comment

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

Did you just copy it from air? Most of these aren't necessary here. Also this is an open source repo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup. Just copied from Air.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also this is an open source repo.

Uff... I was unaware of that fact. So are we at a stage to be keen on backwards compatibility and similar?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tidied up: e1bd8c9

Copy link
Member

Choose a reason for hiding this comment

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

No I'm just saying leaking stuff like filenames or jira numbers might not be a good idea

@stevladimir
Copy link
Contributor Author

As it is a public package (surprise to me) I also scaffolded cabal.project and CI

@stevladimir stevladimir requested a review from mniip April 18, 2025 09:25
@stevladimir stevladimir merged commit 9284156 into master Apr 22, 2025
2 checks passed
@stevladimir stevladimir deleted the stevladimir/tidy-up branch April 22, 2025 09:05
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