webhintio / serverless-online-service Goto Github PK

Now when you create the azure functions has 2 options when you want to use docker containers:

• App service plan: This is the one we are currently using.
• Premium: This one looks to be more prepared for azure functions.

Mostly depends on if we can get a version of Chrome working.

Similar to what we currently have in the online-service.

Look into https://www.terraform.io/ for this.

Original reporter @ingride

It should contain all the different micro-services, queues, how they communicate with each other, how to scale, etc.

We could look into bundle everything with webpack. It should decrease the load time as there will not be hundreds of smaller files. Maybe for another update?

Yep, I think we should create a task for that.

Originally posted by @sarvaje in #6 (comment)

Dependabot encountered the following error when parsing your .dependabot/config.yml:

The property '#/update_configs/0/ignored_updates' of type null did not match the following type: array

Please update the config file to conform with Dependabot's specification using our docs and online validator.

To temporarily workaround an issue with the functions host not starting, I downgraded the FUNCTIONS_EXTENSION_VERSION app setting for our service-functions-production instance to 2.0.13019 instead of ~2.

This should be restored to ~2 at some point after the next update (currently the latest is 2.0.13036). See https://github.com/Azure/azure-functions-host/releases - looks like updates occur every few weeks or less.

If the issue still persists after the next update, we should file a bug against the functions host to ensure it gets investigated.

Right now, if a configuration has a extend, it is not going to work as we expected.

Before creating the job in the database, we create a list of hints with the initial status. If a extends is used, the code is not getting those hints.

The original idea was having everything in Azure function consumption plan, but it was no possible to run chrome headless on it so we were deploying a docker image. Finally, Azure function supports chromium headless, so we should move the worker to consumption plan.

I think this will also allow us to remove the monorepo arquitecture and therefore, simplify the code structure.

src/github/issuereporter.ts(25,22): error TS2709: Cannot use namespace 'Octokit' as a type.
src/github/issuereporter.ts(34,28): error TS2351: This expression is not constructable.
  Type 'typeof import("/home/site/utils/node_modules/@octokit/rest/index")' has no construct signatures.
src/github/issuereporter.ts(51,44): error TS2694: Namespace '"/home/site/utils/node_modules/@octokit/rest/index"' has no exported member 'SearchIssuesResponseItemsItem'.
src/github/issuereporter.ts(61,45): error TS2694: Namespace '"/home/site/utils/node_modules/@octokit/rest/index"' has no exported member 'SearchIssuesResponseItemsItem'.
src/github/issuereporter.ts(68,48): error TS2694: Namespace '"/home/site/utils/node_modules/@octokit/rest/index"' has no exported member 'IssuesUpdateParams'.
src/github/issuereporter.ts(73,22): error TS2694: Namespace '"/home/site/utils/node_modules/@octokit/rest/index"' has no exported member 'IssuesUpdateParams'.
src/github/issuereporter.ts(206,51): error TS7006: Parameter 'label' implicitly has an 'any' type.
src/github/issuereporter.ts(225,52): error TS2694: Namespace '"/home/site/utils/node_modules/@octokit/rest/index"' has no exported member 'SearchIssuesResponseItemsItem'.

/cc @hxlnt @sarvaje

This isn't being used (as it's not configured) and we've removed the associated repo to log issues for failed scans. Time to clean up the code: https://github.com/webhintio/serverless-online-service/blob/main/packages/utils/src/github/issuereporter.ts

Once it is updated, we shouldn't need the script to create the collections.

The configuration for cloudinary is not set in the serverless deployment.

Howdy folks – exploring using webhint for a new effort. As part of that effort exploring what it would take to run natively on AWS.

Looking through the code there are two places Azure specific services are utilized:

• azure-sb
• azure-storage

Would this group be open to accepting contributions that enable multiple vendors for these services?

now, warning is 1 and error is 2, but after the update warning will be 3 and error will be 4

Now this new rule is merged we need to enable this issue in the online scanner.

This requires a new release of webhint.

Everything works fine in local, but in Azure Pipelines tests are failing.

Need to use the Azure Migration tool. It takes a few hours so we will need to do it early in the day.