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

Merge Develop for v0.9.0 #1

Merged
merged 223 commits into from
Sep 28, 2024
Merged

Merge Develop for v0.9.0 #1

merged 223 commits into from
Sep 28, 2024

Conversation

suikan4github
Copy link
Owner

Pre-release version. If no problem, it will become a v1.0.0.

The typedef int i2c_inst_t ; is done inthe picowrapper.hpp now.
The driver lib is now actual lib when it is build to Linux/Win32
Build OK. Not tested.
Stilll under testing. See todo/test_adau1361lower.md
Adau1361Lower.DisablePLL()
Adau1361Lower.InitializeCore()
This is much better than using pre-defined macro by CMAKE file.
TEST_F(Adau1361LowerTest, ConfigureSignalPath)
TEST_F(Adau1361LowerTest, InitializeRegisters)
Explicit note of the initialization sequence.
Move MockI2CMaster to MockI2CMasterInterface.
To refuse the forget to add files.
Now, deploy_pages.yaml calls ctest.yaml explicitly.
(Forgot to add in the previous commit ).
The HTML output is docs/html folder. So, having doc/ is redundant.
docs/duplexslavei2s.md -> docs/i2sslaveduplex.md
To allow doxygen link the class name inside README.md with class explanation,
Remove the explicit link to the sub-md files inside README.md
Update the command to obtain the tools in Ubuntu.
(doxygen and graphviz)

Added the git command to obtain this project and sample project.
To allow the push to branch, we need pages: write.
Forgot to change the GITHUB_TOKEN itself, in the previous commit.
pagesはサイトをビルドするときのみ必要。
These default parameter is not needed becuase this class is
not called from user programmer.
@suikan4github
Copy link
Owner Author

As of 2024/Sep/28, the Doxygen page deployment fails with this error during action:

/home/runner/work/_actions/JamesIves/github-pages-deploy-action/v4/lib/main.js

This is reported as an issue #1697 to the github-pages-deploy-action.

This is not the fail of our code. So, we merge this pull request.

@suikan4github suikan4github merged commit 9741c3b into main Sep 28, 2024
12 of 13 checks passed
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.

1 participant