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

Build features table formatting (bold headers) #1321

Conversation

DeflateAwning
Copy link
Contributor

No description provided.

@barbudor
Copy link
Collaborator

This will not be merged
BUILDS.md and I2CDEVICES.md are sourced from arendst/tasmota
Place your PR there
Thanks

@barbudor barbudor closed this Dec 23, 2023
@DeflateAwning
Copy link
Contributor Author

DeflateAwning commented Dec 23, 2023

Awesome, thank you! Will do that! Did not realize that.

It doesn't seem like the BUILDS.md file is linked or referenced anywhere in the docs repo though. That's why I proposed moving its contents to that file which lists the builds. Any thoughts on that change?

Is the copy automatic? If it is, just out of curiosity, where does that copy action reside? Maybe that copy action could add a markdown comment like:

<!-- This file is automatically copied from https://github.com/arendst/Tasmota/blob/<applicable branch>/BUILDS.md. Please make changes upstream. -->

@barbudor
Copy link
Collaborator

As far as I know the copy is automatic when the docs are built
The doc process master is blakadder

@DeflateAwning
Copy link
Contributor Author

@blakadder, I'm curious if the arendst/Tasmota/BUILDS.md file is copied to this repo (tasmota/docs) automatically or manually? Any thoughts?

@blakadder
Copy link
Contributor

Automatically on each change

@DeflateAwning
Copy link
Contributor Author

@blakadder Do you know where that copy is performed?

Maybe that copy action could add a markdown comment like:

<!-- This file is automatically copied from https://github.com/arendst/Tasmota/blob/<applicable branch>/BUILDS.md. Please make changes upstream. -->

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.

3 participants