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

Added support for new boards #117

Closed
wants to merge 24 commits into from
Closed

Conversation

atcher0
Copy link

@atcher0 atcher0 commented May 31, 2021

@augustofg
Copy link
Member

Hi @atcher0, thank you for the contribution!

We reserve the master branch for stable releases only, please rebase your modifications with our devel branch.

This is a big PR that will need a more careful examination before merging, I already see some conflicts with AFCv4 support code.

@atcher0
Copy link
Author

atcher0 commented Jun 2, 2021

Hi, my changes are now rebased onto the devel branch.

@augustofg augustofg changed the base branch from master to devel June 10, 2021 19:14
@augustofg
Copy link
Member

Unfortunately, this is not possible to merge due to the recent developments of OpenMMC. Support for INA3221 and 8V54816 is already there, so this would generate conflicts. We are migrating away from the GitFlow workflow now, so new PR should be targeting master.

Please open small PRs for each new feature, so it is easier to review and run the CI checks.

@augustofg augustofg closed this Jul 22, 2024
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.

2 participants