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.

Sign up for GitHub

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

Jump to bottom

optimise the CIs #14124

Merged
merged 4 commits into from
Dec 1, 2023
Merged

optimise the CIs #14124

merged 4 commits into from
Dec 1, 2023

Conversation

dhruvinmaniar123
Copy link
Contributor

Updated the CI to skip the tests which are not required.

Describe your changes:

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@dhruvinmaniar123 dhruvinmaniar123 self-assigned this Nov 27, 2023
@github-actions github-actions bot added devops safe to test Add this label to run secure Github workflows on PRs labels Nov 27, 2023
mohityadav766
mohityadav766 previously approved these changes Dec 1, 2023
View reviewed changes
mohityadav766
mohityadav766 previously approved these changes Dec 1, 2023
View reviewed changes
@akash-jain-10 akash-jain-10 merged commit ce77d24 into main Dec 1, 2023
10 checks passed
@akash-jain-10 akash-jain-10 deleted the optimise-CI branch December 1, 2023 11:30
MrVinegar pushed a commit to MrVinegar/OpenMetadata that referenced this pull request Dec 15, 2023
@dhruvinmaniar123 @akash-jain-10
optimise the CIs (open-metadata#14124)
16ede4d 
* optimise the CIs

* fix: Update maven CI workflow commands

* fix: CI Name for maven-build-skip.yaml

---------

Co-authored-by: Akash-Jain <Akash.J@deuexsolutions.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mohityadav766 mohityadav766 mohityadav766 left review comments

@tutte tutte Awaiting requested review from tutte tutte is a code owner

@akash-jain-10 akash-jain-10 Awaiting requested review from akash-jain-10 akash-jain-10 is a code owner

@harshach harshach Awaiting requested review from harshach harshach is a code owner

Assignees

@dhruvinmaniar123 dhruvinmaniar123

Labels
devops safe to test Add this label to run secure Github workflows on PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhruvinmaniar123 @mohityadav766 @akash-jain-10