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

feat: static assets #671

Merged
merged 14 commits into from
Oct 25, 2019
Merged

feat: static assets #671

merged 14 commits into from
Oct 25, 2019

Conversation

P0lip
Copy link
Contributor

@P0lip P0lip commented Oct 11, 2019

Change driven by Studio.
Will describe it in a great detail soon, once it's ready to review.
Long story short - Studio will provide its own assets in the following manner

Spectral.registerStaticAssets(require('@stoplight/spectral/rulesets/assets/assets.json'));

@P0lip P0lip added the WIP label Oct 11, 2019
@P0lip P0lip requested a review from marbemac October 14, 2019 23:38
@P0lip P0lip self-assigned this Oct 14, 2019
@P0lip P0lip added enhancement New feature or request and removed WIP labels Oct 14, 2019
@P0lip P0lip marked this pull request as ready for review October 14, 2019 23:39
@P0lip P0lip force-pushed the feat/assets branch 2 times, most recently from 3b61339 to 8762a87 Compare October 14, 2019 23:50
const target = path.join(baseDir, `assets.json`);
const assets = {};

(async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change needed if we are doing #561 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, it's still going to be needed. It's mostly for Studio to use and we don't expose it anywhere in the docs, etc.
I'll leave a note somewhere in the code on what it actually does and how it works, as right now it's not obvious, and we will forget about why we need it.
I see Windows build is failing, so need to take a look what's broken.

Copy link
Contributor

Choose a reason for hiding this comment

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

@collinbachi did we already switch to whatever new CircleCI payment structure that gets us Windows builds?

Copy link

@collinbachi collinbachi Oct 25, 2019

Choose a reason for hiding this comment

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

We started a 2 week trial of the new structure Wednesday afternoon

Copy link
Contributor

Choose a reason for hiding this comment

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

@P0lip ok, fair enough on the still needed. If you cannot get windows build working with Azure then maybe try out the CircleCI windows build, however that works.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@philsturgeon I got Windows build working, but would love to try out Windows on CircleCI. Azure pipelines is tad slow.

Copy link
Contributor

Choose a reason for hiding this comment

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

Another time! :)

@P0lip P0lip merged commit 475dd2d into develop Oct 25, 2019
@P0lip P0lip deleted the feat/assets branch October 25, 2019 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants