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

Revealjs & update scorecard action version #240

Merged
merged 3 commits into from
Apr 20, 2024
Merged

Revealjs & update scorecard action version #240

merged 3 commits into from
Apr 20, 2024

Conversation

izar
Copy link
Collaborator

@izar izar commented Apr 11, 2024

Added a template to create a https://reveal.js presentation.

Needs tweaks!

@izar izar self-assigned this Apr 11, 2024
@colesmj
Copy link
Collaborator

colesmj commented Apr 12, 2024

Just a note - reveal.js is at https://revealjs.com/

@@ -37,7 +37,7 @@ jobs:
persist-credentials: false

- name: "Run analysis"
uses: ossf/scorecard-action@99c53751e09b9529366343771cc321ec74e9bd3d # v2.0.6
uses: ossf/scorecard-action@v2.3.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this change related to the revealjs addition?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Let me remove this, it should be on an update of its own.

docs/reveal.md Outdated Show resolved Hide resolved
Demo-ing the reveal template.
@colesmj
Copy link
Collaborator

colesmj commented Apr 19, 2024

@izar Did you plan to remove / revert the change to the ossf scorecard? If so I'll hold off on the merge so you can revert that change.

@izar
Copy link
Collaborator Author

izar commented Apr 19, 2024

Nope, it's ok to go forward as is. Must have been some inconsistency between my local repo, most probably some mistake by me in git ops, but 2.3.1 is the right one.

@colesmj colesmj merged commit b0c4388 into master Apr 20, 2024
4 checks passed
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.

2 participants