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

ci: Add GitHub action to upload TICS report to TIOBE #1087

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

bartaz
Copy link
Member

@bartaz bartaz commented May 16, 2024

Done

  • Added GH action workflow to run test coverage and upload TICS report to TIOBE

QA

  • Verify that the test run of this action succeeded
  • You can ignore that the CI job is listed as failed in CI checks on PR

@webteam-app
Copy link

@bartaz bartaz changed the title Add GitHub action to upload TICS report to TIOBE ci: Add GitHub action to upload TICS report to TIOBE May 16, 2024
@bartaz bartaz marked this pull request as ready for review May 16, 2024 09:09
Copy link
Member

@jmuzina jmuzina left a comment

Choose a reason for hiding this comment

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

Looks good, nice work :)

@bartaz bartaz merged commit 394ee8a into main May 16, 2024
13 checks passed
@bartaz bartaz deleted the tiobe-workflow branch May 16, 2024 17:35
@huwshimi
Copy link
Collaborator

I'm curious if you investigated using their GitHub Action: https://github.com/tiobe/tics-github-action#tics-github-action for this or does it not work for our needs?

@bartaz
Copy link
Member Author

bartaz commented May 20, 2024

@huwshimi interesting find. No, I did not. I was following setup instructions I got from TIOBE and previous work @edlerd did for Anbox.

@edlerd Did you try the GH action?

Comment on lines +18 to +23
- name: Lint
run: yarn lint

- name: Build
run: yarn build

Copy link
Collaborator

Choose a reason for hiding this comment

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

We probably could remove these from this job.

@edlerd
Copy link
Contributor

edlerd commented May 21, 2024

@huwshimi interesting find. No, I did not. I was following setup instructions I got from TIOBE and previous work @edlerd did for Anbox.

@edlerd Did you try the GH action?

I tested the GitHub action, when setting up tiobe for lxd-ui. It failed to succeed. As the local run with the bare commands succeeded, I decided to keep them in the workflow going forward.

Copy link

🎉 This PR is included in version 0.53.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants