Merge branch 'master' into master

Author: capricorn86

.github/workflows/release.yml

@@ -4,6 +4,10 @@ on:
   pull_request:
     types: [closed]
 
+permissions:
+  contents: write
+  pull-requests: read
+
 jobs:
   check-next-version:
     runs-on: ubuntu-latest
@@ -105,6 +109,7 @@ jobs:
         run: |
           git config --global user.name "${{ github.actor }}"
           git config --global user.email "$(git log --format='%ae' HEAD^!)"
+          git remote set-url origin https://${{ secrets.GIT_REPOSITORY_URL_USERNAME }}:${{ secrets.GIT_REPOSITORY_URL_ACCESS_TOKEN }}@github.com/capricorn86/happy-dom.git
 
       - name: Creates release branch
         run: |

.npmrc

@@ -0,0 +1 @@
+legacy-peer-deps=true

docs/contributing.md

@@ -6,10 +6,8 @@ We are very happy that you would like to contribute. In this guide you will find
 
 ### Install
 
-We need to add "--legacy-peer-deps" to the install as a workaround to be able to install Vitest. Otherwise we get an "Cannot set properties of null (setting 'parent')" error as Vitest has "happy-dom" as peer dependency. Hopefully we can find a better solution in the future.
-
 ```bash
-npm install --legacy-peer-deps
+npm install
 ```
 
 ### Compile

package-lock.json

@@ -10,7 +10,7 @@ const TIMER = {
  */
 export default class AsyncTaskManager {
 	private static taskID = 0;
-	private runningTasks: { [k: string]: (destroy: boolean) => void } = {};
+	private runningTasks: { [k: string]: (destroy: boolean) => void | Promise<void> } = {};
 	private runningTaskCount = 0;
 	private runningTimers: NodeJS.Timeout[] = [];
 	private runningImmediates: NodeJS.Immediate[] = [];
@@ -189,8 +189,23 @@ export default class AsyncTaskManager {
 			TIMER.clearTimeout(timer);
 		}
 
+		const taskPromises = [];
+
 		for (const key of Object.keys(runningTasks)) {
-			runningTasks[key](destroy);
+			const returnValue = runningTasks[key](destroy);
+			if (returnValue instanceof Promise) {
+				taskPromises.push(returnValue);
+			}
+		}
+
+		if (taskPromises.length) {
+			return Promise.all(taskPromises)
+				.then(() => this.waitUntilComplete())
+				.catch((error) => {
+					/* eslint-disable-next-line no-console */
+					console.error(error);
+					throw error;
+				});
 		}
 
 		// We need to wait for microtasks to complete before resolving.