Skip to content

test: run unit-tests on forks - #29979

Merged
iamsobanjaved merged 2 commits into
openedx:masterfrom
iamsobanjaved:iamsobanjaved/unit-tests-for-forks
Mar 31, 2022
Merged

test: run unit-tests on forks#29979
iamsobanjaved merged 2 commits into
openedx:masterfrom
iamsobanjaved:iamsobanjaved/unit-tests-for-forks

Conversation

@iamsobanjaved

@iamsobanjaved iamsobanjaved commented Feb 25, 2022

Copy link
Copy Markdown
Contributor
  • add verify unit test workflow also
  • use composite GitHub action for syncing

Ran unit tests on my fork here: iamsobanjaved#1

For reference: BOM-3287

Comment thread .github/actions/unit-tests/action.yml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove this and following

@iamsobanjaved
iamsobanjaved marked this pull request as ready for review March 7, 2022 14:00
@iamsobanjaved
iamsobanjaved requested a review from awais786 March 8, 2022 11:14
@iamsobanjaved
iamsobanjaved force-pushed the iamsobanjaved/unit-tests-for-forks branch from 2f143a5 to cb6037f Compare March 18, 2022 07:31
- add verify unit test workflow also
- use composite github action for syncing
@iamsobanjaved
iamsobanjaved force-pushed the iamsobanjaved/unit-tests-for-forks branch from cb6037f to 6ccadbb Compare March 18, 2022 07:33

@Jawayria Jawayria left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As far as composite actions are concerned they are good to go but I don't have much idea about the implementation added to run unit-tests on GitHub Actions

@iamsobanjaved
iamsobanjaved merged commit db201db into openedx:master Mar 31, 2022
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants