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

FUN.md broke when RecordObj changed to HashedRecordObj #1633

Merged
merged 4 commits into from
Mar 19, 2022
Merged

FUN.md broke when RecordObj changed to HashedRecordObj #1633

merged 4 commits into from
Mar 19, 2022

Conversation

MitchellJThomas
Copy link
Contributor

@MitchellJThomas MitchellJThomas commented Mar 18, 2022

The breakage occurred sometime on or before version 1.6

This is a proposed fix for issue #1632

Note: the proposed commit is signed using Level 11 (yes!)

Summary

Switch the jq command to use the HashedRecordObj value

Release Note

Update the instructions for fetching the sig and cert in the FUN.md file. 

@cpanato
Copy link
Member

cpanato commented Mar 18, 2022

@MitchellJThomas please sign the DCO, thanks for this update!

The breakage occured sometime on or before version 1.6

I found it when running this version:
GitVersion:    1.6.0
GitCommit:     4b2c3c0
GitTreeState:  "clean"
BuildDate:     2022-03-03T17:59:06Z
GoVersion:     go1.17.8
Compiler:      gc
Platform:      darwin/amd64

Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>
@MitchellJThomas
Copy link
Contributor Author

MitchellJThomas commented Mar 18, 2022

@MitchellJThomas please sign the DCO, thanks for this update!

Done. You are welcome.

naveensrinivasan and others added 3 commits March 18, 2022 15:52
- OpenSSF Best Practices Badge 94%

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>
The breakage occured sometime on or before version 1.6

I found it when running this version:
GitVersion:    1.6.0
GitCommit:     4b2c3c0
GitTreeState:  "clean"
BuildDate:     2022-03-03T17:59:06Z
GoVersion:     go1.17.8
Compiler:      gc
Platform:      darwin/amd64

Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>
@MitchellJThomas
Copy link
Contributor Author

Seems I kinda goofed with the git rebase HEAD~3 --signoff as it added in commit 9fb9a34. (embarasing)

@codecov-commenter
Copy link

Codecov Report

Merging #1633 (e7b21cc) into main (d0aa73b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1633   +/-   ##
=======================================
  Coverage   27.97%   27.97%           
=======================================
  Files         137      137           
  Lines        7820     7820           
=======================================
  Hits         2188     2188           
  Misses       5403     5403           
  Partials      229      229           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0aa73b...e7b21cc. Read the comment docs.

@dlorenc
Copy link
Member

dlorenc commented Mar 19, 2022

Theres a weird e2e flake that looks to be an internal github error, but this is a doc only change so I'm not going to retry it.

@dlorenc dlorenc merged commit 0e3db34 into sigstore:main Mar 19, 2022
@github-actions github-actions bot added this to the v1.7.0 milestone Mar 19, 2022
@MitchellJThomas MitchellJThomas deleted the broken-fun branch March 19, 2022 17:54
mlieberman85 pushed a commit to mlieberman85/cosign that referenced this pull request May 6, 2022
* FUN.md broke when RecordObj changed to HashedRecordObj

The breakage occured sometime on or before version 1.6

I found it when running this version:
GitVersion:    1.6.0
GitCommit:     4b2c3c0
GitTreeState:  "clean"
BuildDate:     2022-03-03T17:59:06Z
GoVersion:     go1.17.8
Compiler:      gc
Platform:      darwin/amd64

Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>

* Included OpenSSF Best Practices Badge (sigstore#1628)

- OpenSSF Best Practices Badge 94%

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>

* FUN.md broke when RecordObj changed to HashedRecordObj

The breakage occured sometime on or before version 1.6

I found it when running this version:
GitVersion:    1.6.0
GitCommit:     4b2c3c0
GitTreeState:  "clean"
BuildDate:     2022-03-03T17:59:06Z
GoVersion:     go1.17.8
Compiler:      gc
Platform:      darwin/amd64

Signed-off-by: Mitch Thomas <mitch.thomas@gmail.com>

Co-authored-by: Naveen <172697+naveensrinivasan@users.noreply.github.com>
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.

5 participants