Skip to content

Commit

Permalink
Merge branch 'main' into refactor/replace-got/rules-proxy.test.ts
Browse files Browse the repository at this point in the history
  • Loading branch information
hereje authored Dec 7, 2023
2 parents 10dd5a7 + 2626541 commit 3ef1e32
Show file tree
Hide file tree
Showing 8 changed files with 49 additions and 33 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
.delta.dependencyCount
- name: Get TypeScript conversion progress
run:
grep -r --exclude-dir="node_modules" --include="*.mts" "@ts-expect-error" . | wc -l | xargs >
grep -r --exclude-dir="node_modules" --include="*.ts" "@ts-expect-error" . | wc -l | xargs >
.delta.tsConversion && echo " (Number of ts-expect-error directives)" >> .delta.tsConversion
- name: Save PR number
if: github.event_name == 'pull_request'
Expand Down
12 changes: 12 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,18 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).



## [17.10.0](https://github.com/netlify/cli/compare/v17.9.0...v17.10.0) (2023-12-06)


### Features

* add blobs upload step ([#6223](https://github.com/netlify/cli/issues/6223)) ([33177fc](https://github.com/netlify/cli/commit/33177fc0608952196f40249a3e0fae1e42af25a0))


### Bug Fixes

* **deps:** update netlify packages ([#6244](https://github.com/netlify/cli/issues/6244)) ([b3f857a](https://github.com/netlify/cli/commit/b3f857a939ed78e726545092c2bbb2d80ae5105e))

## [17.9.0](https://github.com/netlify/cli/compare/v17.8.1...v17.9.0) (2023-12-04)


Expand Down
32 changes: 16 additions & 16 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "netlify-cli",
"description": "Netlify command line tool",
"version": "17.9.0",
"version": "17.10.0",
"author": "Netlify Inc.",
"type": "module",
"engines": {
Expand Down Expand Up @@ -81,8 +81,8 @@
"@bugsnag/js": "7.20.2",
"@fastify/static": "6.10.2",
"@netlify/blobs": "6.3.1",
"@netlify/build": "29.29.4",
"@netlify/build-info": "7.11.1",
"@netlify/build": "29.31.0",
"@netlify/build-info": "7.11.3",
"@netlify/config": "20.10.0",
"@netlify/edge-bundler": "10.1.3",
"@netlify/local-functions-proxy": "1.1.1",
Expand Down
22 changes: 13 additions & 9 deletions tests/integration/commands/deploy/deploy.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -824,11 +824,15 @@ describe.skipIf(process.env.NETLIFY_TEST_DISABLE_LIVE === 'true').concurrent('co
await builder
.withNetlifyToml({
config: {
build: { publish: 'dist', functions: 'functions' },
build: { functions: 'functions', publish: 'dist' },
},
})
.withContentFile({
path: 'dist/.netlify/blobs/deploy/hello',
path: 'dist/index.html',
content: '<a href="/read-blob">get blob</a>',
})
.withContentFile({
path: '.netlify/blobs/deploy/hello',
content: 'hello from the blob',
})
.withPackageJson({
Expand All @@ -843,21 +847,21 @@ describe.skipIf(process.env.NETLIFY_TEST_DISABLE_LIVE === 'true').concurrent('co
path: 'functions/read-blob.ts',
content: `
import { getDeployStore } from "@netlify/blobs"
import { Config, Context } from "@netlify/functions"
export default async (req: Request, context: Context) => {
import { Config } from "@netlify/functions"
export default async () => {
const store = getDeployStore()
const blob = await store.get('hello')
return new Response(blob)
}
export const config: Config = {
path: "/read-blob"
}
}
`,
})
.buildAsync()
.build()

await execa.command('npm install', { cwd: builder.directory })
const { deploy_url: deployUrl } = await callCli(
Expand Down
4 changes: 2 additions & 2 deletions tests/integration/commands/dev/dev-miscellaneous.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -895,7 +895,7 @@ describe.concurrent('commands/dev-miscellaneous', () => {
})
})

test('should respect in-source configuration from internal edge functions', async (t) => {
test.skip('should respect in-source configuration from internal edge functions', async (t) => {
await withSiteBuilder('site-with-internal-edge-functions', async (builder) => {
const publicDir = 'public'
await builder
Expand Down Expand Up @@ -948,7 +948,7 @@ describe.concurrent('commands/dev-miscellaneous', () => {
})
})

test('Serves edge functions with import maps coming from the `functions.deno_import_map` config property and from the internal manifest', async (t) => {
test.skip('Serves edge functions with import maps coming from the `functions.deno_import_map` config property and from the internal manifest', async (t) => {
await withSiteBuilder('site-with-edge-functions-and-import-maps', async (builder) => {
const internalEdgeFunctionsDir = path.join('.netlify', 'edge-functions')

Expand Down
2 changes: 1 addition & 1 deletion tests/integration/commands/dev/edge-functions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ const setup = async ({ fixture }) => {

describe.skipIf(isWindows)('edge functions', () => {
setupFixtureTests('dev-server-with-edge-functions', { devServer: true, mockApi: { routes } }, () => {
test<FixtureTestContext>('should run edge functions in correct order', async ({ devServer }) => {
test.skip<FixtureTestContext>('should run edge functions in correct order', async ({ devServer }) => {
const response = await got(`http://localhost:${devServer.port}/ordertest`, {
throwHttpErrors: false,
retry: { limit: 0 },
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,7 @@ exports.handler = async () => ({
})
})

test('Serves functions from the internal functions directory', async (t) => {
test.skip('Serves functions from the internal functions directory', async (t) => {
await withSiteBuilder('function-internal', async (builder) => {
const bundlerConfig = args.includes('esbuild') ? { node_bundler: 'esbuild' } : {}

Expand Down

0 comments on commit 3ef1e32

Please sign in to comment.