Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Add Github Actions #47

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add Github Actions #47

wants to merge 2 commits into from

Conversation

BoisterousCoder
Copy link
Contributor

Description

This adds Github actions to the repo

Link to issue

closes #38

Type of change

  • New feature (non-breaking change that adds functionality)

@BoisterousCoder BoisterousCoder added the enhancement New feature or request label Jan 4, 2023
@BoisterousCoder BoisterousCoder requested a review from bgins January 4, 2023 20:02
@BoisterousCoder BoisterousCoder self-assigned this Jan 4, 2023
@BoisterousCoder BoisterousCoder linked an issue Jan 4, 2023 that may be closed by this pull request
@@ -0,0 +1,58 @@
name: 📈 Benchmark

Choose a reason for hiding this comment

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

can probably avoid this until we get benches into the code?

Copy link

@zeeshanlakhani zeeshanlakhani left a comment

Choose a reason for hiding this comment

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

good start. There are a few actions that are unnecessary, and there's some templating from rust-template thrown-in which is specific to cargo-generate and not related to direct .yml.

@@ -0,0 +1,117 @@
name: 🐳 Docker

Choose a reason for hiding this comment

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

can also remove this ftm

command: manifest
extra-files: |
README.md
{% if auditable %}

Choose a reason for hiding this comment

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

these are copied from the template for generation and won't work in this context.

@BoisterousCoder
Copy link
Contributor Author

That should be the removal of all that template stuff and benchmark stuff. Have fun!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Github Actions
2 participants