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

Feed sync #456

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Feed sync #456

wants to merge 13 commits into from

Conversation

syspect-sec
Copy link
Contributor

@syspect-sec syspect-sec commented Mar 7, 2024

What

The comprehensive guide to new and legacy feed sync tools is now available.

Why

Provide users with a comprehensive guide to Greenbone feed sync tooling

Checklist

  • entry added to changelog.md
  • most recent changes to greenbone/docs main branch have been merged

@syspect-sec syspect-sec requested a review from a team as a code owner March 7, 2024 05:13
@greenbonebot greenbonebot enabled auto-merge (rebase) March 7, 2024 05:13
@syspect-sec
Copy link
Contributor Author

syspect-sec commented Mar 8, 2024

Although the auto merge fails (Build the Greenbone Documentation / Build Documentation (pull_request) ) due to during the auto build, myst reports an error: /home/runner/work/docs/docs/src/feed-sync.md:37: WARNING: 'myst' cross-reference target not found: 'available-feed-types' [myst.xref_missing], the upon second compile, the docs build fine and even after the first, build, (and second) the link is active and works to redirect to the correct location.

Interestingly, although the Greenbone docs use the same syntax for inline, same page links in other places, I cannot find any reference for using the [link-text](#header-title) method on either the commonmark.org markdown tutorial, or the recommended syntax guide. Although, the [link-text](#header-title) method seems to use the standard HTML internal linking syntax to headers by name, so I guess that's why it works even though it's not officially supported?

Also, I have tracked down the issue to seems to be all fourth level header links, and I've opened an issue on the MySTParser GitHub page.

@cfi-gb
Copy link
Member

cfi-gb commented Mar 27, 2024

Note that the deprecated / old greenbone-nvt-sync has been now removed completely in the scanner with / since greenbone/openvas-scanner#1589 so some adjustments to this PR might be required.

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.

None yet

3 participants