Update to node v7 builder #482
Conversation
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
![semgrep-code-vtex-apps[bot]](https://avatars.githubusercontent.com/u/12614072?s=60&v=4){kind=small}
| @@ -6,7 +6,7 @@ | |||
| "description": "GraphQL resolvers for the VTEX API for the catalog and orders.", | |||
| "credentialType": "absolute", | |||
| "builders": { | |||
| "node": "6.x", | |||
| expect(tail.index).toBe(1) | ||
| expect(tail.queryArray).toMatchObject(expectedQueryArray) | ||
| expect(tail.mapArray).toMatchObject(expectedMapArray) | ||
| expect(mockContext.clients.rewriter.getRoute).toBeCalledTimes(2) | ||
| }) | ||
| }) | ||
|
|
||
| describe('tests related to the productSearch query', () => { | ||
| describe.skip('tests related to the productSearch query', () => { |
There was a problem hiding this comment.
Skiping theese tests because they were already broken and to fix it I would need to make this PR even bigger...
|
Your PR has been merged! App is being published. 🚀 After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:
After that your app will be updated on all accounts. For more information on the deployment process check the docs. 📖 |
What problem is this solving?
How should this be manually tested?
Workspace
Checklist/Reminders
README.md.CHANGELOG.md.Screenshots or example usage
Type of changes
Notes