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

Chore/sof 5945 #83

Merged
merged 53 commits into from
Aug 11, 2023
Merged

Chore/sof 5945 #83

merged 53 commits into from
Aug 11, 2023

Conversation

azech-hqs
Copy link
Contributor

This PR concerns:

  • migrating static data from JS to Yaml

@@ -0,0 +1,93 @@
name: espresso
Copy link
Member

Choose a reason for hiding this comment

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

Let's add a YAML linter for the PRs pls, @chu-k already added for https://github.com/Exabyte-io/python_code/blob/main/.github/workflows/cicd.yml

- version: "6.7.0"
isDefault: true
hasAdvancedComputeOptions: true
# version 6.8.0
Copy link
Member

Choose a reason for hiding this comment

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

maybe #### version 6.8.0 #### or something like that to make it more pronounced

@@ -0,0 +1,142 @@
// eslint-disable
Copy link
Member

Choose a reason for hiding this comment

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

Let's minimize the autogenerated files, pls

with:
repository: Exabyte-io/actions
token: ${{ secrets.BOT_GITHUB_TOKEN }}
ref: 4d3e0fc415f6eb28739a331e7f44cd7ba242668f
Copy link
Member

Choose a reason for hiding this comment

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

Can be omitted, actions PR is merged now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@azech-hqs azech-hqs merged commit a9b232f into dev Aug 11, 2023
7 checks passed
@azech-hqs azech-hqs deleted the chore/SOF-5945 branch August 11, 2023 19:25
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