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

Ensure CI runs on push to main #414

Merged
merged 1 commit into from
Sep 20, 2024
Merged

Ensure CI runs on push to main #414

merged 1 commit into from
Sep 20, 2024

Conversation

glopesdev
Copy link
Contributor

@glopesdev glopesdev commented Sep 20, 2024

Adds a CI workflow trigger on push to main to ensure the action runs when merging pull requests. This should also enable the CI badge on the main README file.

Summary by CodeRabbit

  • Chore: Implemented a Continuous Integration (CI) workflow. This update will automatically trigger the build environment and run tests whenever changes are pushed to the main branch. This ensures that all new code is properly tested, improving the reliability of our software for end-users.

@glopesdev glopesdev added the feature New planned feature label Sep 20, 2024
Copy link

github-actions bot commented Sep 20, 2024

Image description CodeRabbit

Walkthrough

The overall change involves the addition of a Continuous Integration (CI) workflow that triggers on push to the main branch. This workflow is designed to build the environment and run tests automatically, ensuring code quality and consistency with every push made to the main branch.

Changes

File Summary
.github/workflows/build_env_run_tests.yml Added a CI workflow trigger on push to the main branch.

Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between a889dba and 2fb1058 commits.
Files selected (1)
  • .github/workflows/build_env_run_tests.yml (1)
Files skipped from review due to trivial changes (1)
  • .github/workflows/build_env_run_tests.yml
Review comments generated (0)
  • Review: 0
  • LGTM: 0

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

Copy link

codecov bot commented Sep 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 15.81%. Comparing base (1de5c25) to head (2fb1058).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #414   +/-   ##
=======================================
  Coverage   15.81%   15.81%           
=======================================
  Files          37       37           
  Lines        3009     3009           
=======================================
  Hits          476      476           
  Misses       2533     2533           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glopesdev glopesdev merged commit 7812b4f into main Sep 20, 2024
6 checks passed
@glopesdev glopesdev deleted the gl-ruff-check branch September 20, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New planned feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants