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

Update readme #177

Merged
merged 10 commits into from
Apr 23, 2024
Merged

Update readme #177

merged 10 commits into from
Apr 23, 2024

Conversation

pitdicker
Copy link
Contributor

No description provided.

parse-zoneinfo/README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@@ -14,8 +14,6 @@ Parse-zoneinfo is a fork of [`zoneinfo_parse`] by Benjamin Sago (now unmaintaine
[`zoneinfo_parse`]: https://crates.io/crates/zoneinfo_parse
[`chrono-tz`]: https://crates.io/crates/chrono-tz

### [View the Rustdoc](https://docs.rs/parse-zoneinfo)
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably add back a bunch of badges like we have in the chrono README. Having a link to the docs is actually useful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is something I also wanted to see and forgot about! I'll open a fresh PR for that.


The `example` folder of this crate contains a pasic parser example.
Copy link
Contributor

Choose a reason for hiding this comment

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

"pasic" -> "basic"


## Example program
Parse-zoneinfo is used to produce the data for [`chrono-tz`]. For a full example of its use see [`chrono-tz-build`].
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's lowercase "parse-zoneinfo" here?

parse-zoneinfo/README.md Show resolved Hide resolved
@pitdicker pitdicker added this pull request to the merge queue Apr 23, 2024
Merged via the queue into chronotope:main with commit 3ace57d Apr 23, 2024
4 checks passed
@pitdicker pitdicker deleted the update_readme branch April 23, 2024 11:57
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

2 participants