Fix/typescript tests #5375

DigitalBrainJS · 2022-12-14T18:43:55Z

Added release:no-npm script
Fixed index.d.ts types (mainly headers types)
Refactored typescripts tests
- Moved to module's entry point tests
- Used tsc --noEmit instead of dtslint

� Conflicts: � package-lock.json � package.json

… making an npm release;

test/module/test.js

+        await spawn(`npm test --prefix ${pkgPath}`, [], {
+          shell : true,
+          stdio: 'pipe'
+        });


test/module/test.js

+        await spawn(`npm test --prefix ${pkgPath}`, [], {
+          shell : true,
+          stdio: 'pipe'
+        });


test/module/typings/cjs/index.ts

-  });
+    .catch((error) => {
+      if (axios.isAxiosError(error)) {
+        const axiosError: axios.AxiosError = error;


test/module/typings/esm/index.ts

refactor(test): refactor typescript tests;

test/module/typings/esm/index.ts

+
+// issue #5034
+
+function getRequestConfig1(options: AxiosRequestConfig): AxiosRequestConfig {


test/module/typings/esm/index.ts

+  };
+}
+
+function getRequestConfig2(options: AxiosRequestConfig): AxiosRequestConfig {


DigitalBrainJS added 3 commits December 10, 2022 19:43

chore(ci): Add release-it script;

3f2f496

Merge branch 'v1.x' of https://github.com/axios/axios into Axios-1.x

e106c04

� Conflicts: � package-lock.json � package.json

chore(ci): add release:no-npm script to release the package without…

c069d26

… making an npm release;

github-advanced-security bot found potential problems Dec 14, 2022

View reviewed changes

DigitalBrainJS force-pushed the fix/typescript-tests branch from a86a669 to fb8e289 Compare December 14, 2022 19:37

github-advanced-security bot found potential problems Dec 14, 2022

View reviewed changes

fix(test): fixed d.ts typings;

7e02f5f

refactor(test): refactor typescript tests;

DigitalBrainJS force-pushed the fix/typescript-tests branch from fb8e289 to 7e02f5f Compare December 14, 2022 20:21

github-advanced-security bot found potential problems Dec 14, 2022

View reviewed changes

DigitalBrainJS merged commit 7a2f901 into axios:v1.x Dec 14, 2022
7 checks passed

karlhorky mentioned this pull request Dec 30, 2022

Property 'Authorization' does not exists on type 'AxiosHeaders' after upgrade to 1.2.2 #5416

Open


		// issue #5034

		function getRequestConfig1(options: AxiosRequestConfig): AxiosRequestConfig {

Fix/typescript tests #5375

Fix/typescript tests #5375

Conversation

DigitalBrainJS commented Dec 14, 2022 • edited

DigitalBrainJS commented Dec 14, 2022 •

edited