Skip to content

check if the link exists in readthedocs. Fixes #447 - #1080

Merged
dgarijo merged 4 commits into
KnowledgeCaptureAndDiscovery:devfrom
juanjemdIos:fix-447
Aug 24, 2026
Merged

check if the link exists in readthedocs. Fixes #447#1080
dgarijo merged 4 commits into
KnowledgeCaptureAndDiscovery:devfrom
juanjemdIos:fix-447

Conversation

@juanjemdIos

Copy link
Copy Markdown
Contributor

No description provided.

juanjemdIos and others added 3 commits August 19, 2026 09:24
…h flag. But now it is more robust. Fixes KnowledgeCaptureAndDiscovery#510.

Note: Mock tests were written with AI helped. Some mockups are still too complex for me to build on my own.
Comment thread src/somef/somef_cli.py Outdated
repo_type = constants.RepositoryType.GITHUB
repository_metadata = Result()
def_branch = "main"
def_branch = "master"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thee default branch is not master. It is main. This is incorrect.

@dgarijo dgarijo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please see the comments

@juanjemdIos

Copy link
Copy Markdown
Contributor Author

so... the defautl branch should be 'main'? I changed that because in some places we had main and in other master. I tried to unify the code.

@dgarijo

dgarijo commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Right now the default branch is always main in new repos. So the default should be main, the fallback is master.

@dgarijo
dgarijo merged commit 2422b0d into KnowledgeCaptureAndDiscovery:dev Aug 24, 2026
2 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.

2 participants