catalystneuro / format-support-table

A summary of format support in the NWB ecosystem
https://catalystneuro.github.io/format-support-table/
1 stars 1 forks source link

Create test-pull-request-pages.yml #24

Closed CodyCBakerPhD closed 1 year ago

CodyCBakerPhD commented 1 year ago

@garrettmflynn Hoping this makes it easier to test changes on PRs

garrettmflynn commented 1 year ago

Where do you visit to review the preview files?

CodyCBakerPhD commented 1 year ago

If it was working it would pop up with a link via the CI

garrettmflynn commented 1 year ago

Got it. And we can't run without merging in main, right?

I'm a bit skeptical of the _site targeting—though that might be fine given the actions you're working with. When I've seen that subfolder before, you have to ensure that files are built there, while we're usually just targeting the main folder.

Can always adjust in a follow-up.

CodyCBakerPhD commented 1 year ago

Ah, you're right - this was taken from https://github.com/NeurodataWithoutBorders/nwb_hackathons where they keep the built version on a different branch

CodyCBakerPhD commented 1 year ago

If you know a better way to test deployments on PRs let me know - I'd just like a way to test things before merging

github-actions[bot] commented 1 year ago

PR Preview Action v1.4.4 :---: :rocket: Deployed preview to https://catalystneuro.github.io/format-support-table/pr-preview/pr-24/ on branch gh-pages at 2023-10-17 15:53 UTC