Skip to content

Conversation

@seddonym
Copy link
Collaborator

@seddonym seddonym commented Sep 12, 2024

Set up a few basics for the project - dev requirements, docs, license, github actions etc.

What this builds on

Builds on this commit, which is at the time of writing the only commit on the project.

That commit was generating using using cookiecutter gh:octoenergy/cookiecutter-kraken-package.

I changed one small thing before committing it: I renamed the package from kraken.rustfluent just to rustfluent as it didn't feel the namespace was appropriate in this case.

Copy link
Collaborator Author

@seddonym seddonym Sep 12, 2024

Choose a reason for hiding this comment

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

This is just following the instructions in the cookie cutter template. The project doesn't have any production package dependencies so there's no need to do matrix testing along those lines.

@seddonym seddonym changed the title Set up pre-commit Set up project Sep 12, 2024
@@ -0,0 +1,189 @@
# Contributing
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

All of these docs are not relevant to consumers of the package, only to contributors.

@seddonym seddonym marked this pull request as ready for review September 12, 2024 14:23
Copy link

@khorn khorn left a comment

Choose a reason for hiding this comment

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

Tiny typo, otherwise 👍

README.md Outdated
TODO describe the purpose of 'rustfluent' here.
A Python interface to the Rust Fluent Library.

This project is a shall shim around [fluent-rs](https://github.com/projectfluent/fluent-rs), so it
Copy link

Choose a reason for hiding this comment

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

Typo: 'shall' -> "small"

seddonym and others added 2 commits September 13, 2024 09:14
Co-authored-by: James Tunnicliffe <james.tunnicliffe@nanosheep.org>
@seddonym seddonym merged commit e471852 into main Sep 13, 2024
1 check passed
@seddonym seddonym deleted the setup branch September 13, 2024 08:20
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.

3 participants