Skip to content

Conversation

@omkar-334
Copy link
Contributor

@omkar-334 omkar-334 commented Jul 23, 2025

@aatmanvaidya this workflow should be triggered on commits to main branch only right?

Setting up the documentation and /feluda route has to be done in the repository settings by the maintainers.
Resolves issue #649

cc @dennyabrain

@omkar-334 omkar-334 changed the base branch from main to development July 23, 2025 18:20
@aatmanvaidya aatmanvaidya self-requested a review July 23, 2025 18:49
@aatmanvaidya
Copy link
Collaborator

@omkar-334 yes it should be triggered when code is merged to the main branch, but for now also do add development branch. because before we push everything to main, it would be good to once deploy docs and see everything

@aatmanvaidya
Copy link
Collaborator

also @omkar-334 in reference to this

Setting up the documentation and /feluda route has to be done in the repository settings by the maintainers.

can you link any articles/docs from github that we can follow in configuring repo settings?

@omkar-334
Copy link
Contributor Author

can you link any articles/docs from github that we can follow in configuring repo settings?

image The branch should be `gh-pages` and the route should be `/feluda`

@omkar-334
Copy link
Contributor Author

https://docs.github.com/en/pages/quickstart - here's the docs by github

@aatmanvaidya
Copy link
Collaborator

thank you this is super helpful

@dennyabrain
Copy link
Contributor

I have a handle on the repo configuration..once the build folder with the index.html is on GitHub, let me know and I'll do the rest

@aatmanvaidya aatmanvaidya merged commit 2e637bb into tattle-made:development Jul 24, 2025
3 of 4 checks passed
@aatmanvaidya
Copy link
Collaborator

@dennyabrain the build folder has been pushed to the gh-pages branch and there is a index.html file also in it - https://github.com/tattle-made/feluda/tree/gh-pages

@omkar-334 omkar-334 deleted the publish branch July 24, 2025 04:34
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