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

chore(gatsby): migrate bootstrap load-themes to typescript #32786

Conversation

nategiraudeau
Copy link
Contributor

Description

Migrates packages/gatsby/src/bootstrap/load-themes to TypeScript.

Related Issues

Related to #21995

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 12, 2021
@nategiraudeau nategiraudeau changed the title migrate bootstrap load-themes to typescript chore(gatsby): migrate bootstrap load-themes to typescript Aug 12, 2021
@gatsbot
Copy link

gatsbot bot commented Aug 12, 2021

Fails
🚫

Danger failed to run /app/peril-downloaded-gatsbyjs/gatsby@peril/rules/validate-yaml.ts.

Error TypeError

Cannot read property 'modified_files' of undefined
TypeError: Cannot read property 'modified_files' of undefined
    at Promise.all.Object.entries.map (/app/peril-downloaded-gatsbyjs/gatsby@peril/rules/validate-yaml.ts:149:25)
    at Array.map (<anonymous>)
    at Object.exports.validateYaml (/app/peril-downloaded-gatsbyjs/gatsby@peril/rules/validate-yaml.ts:148:52)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Dangerfile

144|         twitter: Joi.string().regex(/^@/),
145|       })
146|     )
147|     .unique("id")
148| }
----------------------------^
149| 
150| const getStartersSchema = categories => {
151|   return Joi.array()
152|     .items(

Generated by 🚫 dangerJS

@LekoArts LekoArts removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 16, 2021
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.

2 participants