-
Notifications
You must be signed in to change notification settings - Fork 161
fix(spec): move specification to root dir, symlink to public #633
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for webmonetization-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
2bcfc7c to
a3498a1
Compare
a3498a1 to
7b7ba03
Compare
public/specification
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JoblersTune Please check this continues to work on Windows. I've tested on MacOS and Linux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately I don't have a Windows machine either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe one of the tech writers can help, if I remember one of them had a Windows machine, maybe @brad-dow ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DarianM Can you check the follow work ok:
pnpm startpnpm build; pnpm preview
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, let me do it asap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😭 Can you check on /specification/ as well (note: trailing slash)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing slash has the same behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/specification preview works fine, there's a /specification 404 dev problem for windows. bun build && bun preview are also fine; using pnpm fine too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1d50946 should fix dev as well.
Move specs to root dir, making it easier to discover and update, separate from Astro structure.
Symlink it to public/specification (at build time for platform compatibility), so Astro copies the specifications as is without any processing (including all the includes), so ReSpec documents continue to work.
cc: @lukewarlow @asurkov
Before:
After: