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

Add utility to test doc-website templates #4263

Merged
merged 1 commit into from
Mar 19, 2023
Merged

Add utility to test doc-website templates #4263

merged 1 commit into from
Mar 19, 2023

Conversation

coke
Copy link
Collaborator

@coke coke commented Mar 11, 2023

The problem

templates in doc-website exist but are not verified to pass raku/doc's basic pod structure.

Solution provided

util/test-website

@cfa
Copy link
Contributor

cfa commented Mar 12, 2023

Hmm, shouldn't this be tested in the website repo itself?

(Not sure why we'd want to mix content and presentation again.)

@coke
Copy link
Collaborator Author

coke commented Mar 12, 2023

If we keep the tests in raku/doc and the templates in doc-website, what's the advantage of running the tests from either area? Either way we have to cover the other repo, yes?

@cfa
Copy link
Contributor

cfa commented Mar 13, 2023

Just that this requires cloning the other repo in order to run tests on its content.

@coke
Copy link
Collaborator Author

coke commented Mar 13, 2023

And in the other case, we clone this repo to get the tests to run.

@coke coke marked this pull request as ready for review March 14, 2023 17:17
@coke
Copy link
Collaborator Author

coke commented Mar 19, 2023

So, since either way we have to do a clone to get the tests and the files in the same place, let's go with this one for now.

@coke coke merged commit d7ca402 into main Mar 19, 2023
@coke coke deleted the test-web-docs branch March 19, 2023 02:39
@finanalyst
Copy link
Collaborator

see Raku/doc-website#103

coke added a commit that referenced this pull request Jul 30, 2023
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