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

Add new action to update leap seconds automatically #1583

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

AurelienFT
Copy link
Contributor

Resolves : #675

This PR adds a github actions that runs each week, to check for the leap seconds change in a reputable source and creates a PR to update if necessary.

The PR doesn't contains the process to release automatically as the process could change and maybe we don't want to directly change the version and make the release.

This current PR also include the changes to start with the current value 37 seconds.
Here is some tests I did on a fork : https://github.com/AurelienFT/formats/actions/runs/11470018305/job/31918401654 / https://github.com/AurelienFT/formats/pull/5/files

It could be nice to release a new major this time to have the correct time available for everyone.

@tarcieri
Copy link
Member

I think we should just put out patch releases that correct the time offset

@tarcieri tarcieri merged commit 0bd9dea into RustCrypto:master Oct 23, 2024
10 checks passed
@AurelienFT
Copy link
Contributor Author

AurelienFT commented Oct 23, 2024

@tarcieri Thanks a lot for the fast merge. Can we expect a release for the first update that has been done in this PR ? :)

@AurelienFT
Copy link
Contributor Author

@tarcieri Hello, any news about a possible release for this fix ?

baloo added a commit to baloo/formats that referenced this pull request Oct 29, 2024
Changed
- bump zeroize from 1.4.2 to 1.4.3 (RustCrypto#187)
- Lint improvements (RustCrypto#824)
- bump zeroize from 1.5.7 to 1.6.0 (RustCrypto#956)
- bump zeroize from 1.7.0 to 1.8.1 (RustCrypto#1419)

Fixed
- fix TAI offset and verify with GH Action (RustCrypto#1583)
@baloo baloo mentioned this pull request Oct 29, 2024
baloo added a commit to baloo/formats that referenced this pull request Oct 29, 2024
Fixed
- fix TAI offset and verify with GH Action (RustCrypto#1583)
@baloo baloo mentioned this pull request Oct 29, 2024
baloo added a commit to baloo/formats that referenced this pull request Oct 29, 2024
Fixed
- fix TAI offset and verify with GH Action (RustCrypto#1583)
@baloo baloo mentioned this pull request Oct 29, 2024
@tarcieri
Copy link
Member

#1591 is open but it's been a bit tricky because of how long it's been since a release

baloo added a commit to baloo/formats that referenced this pull request Oct 29, 2024
Fixed
- fix TAI offset and verify with GH Action (RustCrypto#1583)
baloo added a commit that referenced this pull request Oct 29, 2024
Fixed
- fix TAI offset and verify with GH Action (#1583)
AurelienFT added a commit to FuelLabs/fuel-core that referenced this pull request Oct 30, 2024
## Linked Issues/PRs
#2338
RustCrypto/formats#1583
RustCrypto/formats#1590

## Description
I have fixed the wrong offset and added an automation for future update
in crate `tai64`(RustCrypto/formats#1583) and
they merged it and made a new release
(RustCrypto/formats#1590). This PR updates to
the new version.

## Checklist
- [x] Breaking changes are clearly marked as such in the PR description
and changelog
- [x] New behavior is reflected in tests
- [x] [The specification](https://github.com/FuelLabs/fuel-specs/)
matches the implemented behavior (link update PR if changes are needed)

### Before requesting review
- [x] I have reviewed the code myself
- [x] I have created follow-up issues caused by this PR and linked them
here
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants