Skip to content

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 5, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
flow-bin (changelog) ^0.95.1 -> ^0.269.0 age adoption passing confidence

Release Notes

flowtype/flow-bin (flow-bin)

v0.269.1

Compare Source

v0.268.0

Compare Source

v0.267.0

Compare Source

v0.266.1

Compare Source

v0.266.0

Compare Source

v0.265.3

Compare Source

v0.265.2

Compare Source

v0.265.1

Compare Source

v0.265.0

Compare Source

v0.264.0

Compare Source

v0.263.0

Compare Source

v0.262.0

Compare Source

v0.261.2

Compare Source

v0.261.1

Compare Source

v0.261.0

Compare Source

v0.260.0

Compare Source

v0.259.1

Compare Source

v0.259.0

Compare Source

v0.258.1

Compare Source

v0.258.0

Compare Source

v0.257.1

Compare Source

v0.257.0

Compare Source

v0.256.0

Compare Source

v0.255.0

Compare Source

v0.254.2

Compare Source

v0.254.1

Compare Source

v0.254.0

Compare Source

v0.253.0

Compare Source

v0.252.0

Compare Source

v0.251.1

Compare Source

v0.251.0

Compare Source

v0.250.0

Compare Source

v0.249.0

Compare Source

v0.248.1

Compare Source

v0.248.0

Compare Source

v0.247.1

Compare Source

v0.247.0

Compare Source

v0.246.0

Compare Source

v0.245.2

Compare Source

v0.245.1

Compare Source

v0.245.0

Compare Source

v0.244.0

Compare Source

v0.243.0

Compare Source

v0.242.1

Compare Source

v0.242.0

Compare Source

v0.241.0

Compare Source

v0.240.0

Compare Source

v0.239.1

Compare Source

v0.239.0

Compare Source

v0.238.3

Compare Source

v0.238.2

Compare Source

v0.238.1

Compare Source

v0.238.0

Compare Source

v0.237.2

Compare Source

v0.237.1

Compare Source

v0.237.0

Compare Source

v0.236.0

Compare Source

v0.235.1

Compare Source

v0.234.0

Compare Source

v0.233.0

Compare Source

v0.232.0

Compare Source

v0.231.0

Compare Source

v0.230.0

Compare Source

v0.229.2

Compare Source

v0.229.0

Compare Source

v0.228.0

Compare Source

v0.227.0

Compare Source

v0.226.0

Compare Source

v0.225.1

Compare Source

v0.225.0

Compare Source

v0.224.0

Compare Source

v0.223.3

Compare Source

v0.223.2

Compare Source

v0.223.0

Compare Source

v0.222.0

Compare Source

v0.221.0

Compare Source

v0.220.1

Compare Source

v0.220.0

Compare Source

v0.219.5

Compare Source

v0.219.4

Compare Source

v0.219.3

Compare Source

v0.219.2

Compare Source

v0.219.0

Compare Source

v0.218.1

Compare Source

v0.218.0

Compare Source

v0.217.2

Compare Source

v0.217.1

Compare Source

v0.217.0

Compare Source

v0.216.1

Compare Source

v0.216.0

Compare Source

v0.215.1

Compare Source

v0.215.0

Compare Source

v0.214.0

Compare Source

v0.213.1

Compare Source

v0.213.0

Compare Source

v0.212.0

Compare Source

v0.211.1

Compare Source

v0.211.0

Compare Source

v0.210.2

Compare Source

v0.210.1

Compare Source

v0.210.0

Compare Source

v0.209.0

Compare Source

v0.208.1

Compare Source

v0.208.0

Compare Source

v0.207.0

Compare Source

v0.206.0

Compare Source

v0.205.1

Compare Source

v0.205.0

Compare Source

v0.204.1

Compare Source

v0.204.0

Compare Source

v0.203.1

Compare Source

v0.203.0

Compare Source

v0.202.1

Compare Source

v0.202.0

Compare Source

v0.201.0

Compare Source

v0.200.1

Compare Source

v0.200.0

Compare Source

v0.199.1

Compare Source

v0.199.0

Compare Source

v0.198.2

Compare Source

v0.198.1

Compare Source

v0.198.0

Compare Source

v0.197.0

Compare Source

v0.196.3

Compare Source

v0.196.2

Compare Source

v0.196.1

Compare Source

v0.196.0

Compare Source

v0.195.2

Compare Source

v0.195.1

Compare Source

v0.195.0

Compare Source

v0.194.0

Compare Source

v0.193.0

Compare Source

v0.192.0

Compare Source

v0.191.0

Compare Source

v0.190.1

Compare Source

v0.190.0

Compare Source

v0.189.0

Compare Source

v0.188.2

Compare Source

v0.188.1

Compare Source

v0.188.0

Compare Source

v0.187.1

Compare Source

v0.187.0

Compare Source

v0.186.0

Compare Source

v0.185.2

Compare Source

v0.185.1

Compare Source

v0.185.0

Compare Source

v0.184.0

Compare Source

v0.183.1

Compare Source

v0.183.0

Compare Source

v0.182.0

Compare Source

v0.181.2

Compare Source

v0.181.1

Compare Source

v0.181.0

Compare Source

v0.180.1

Compare Source

v0.180.0

Compare Source

v0.179.0

Compare Source

v0.178.1

Compare Source

v0.178.0

Compare Source

v0.177.0

Compare Source

v0.176.3

Compare Source

v0.176.2

Compare Source

v0.176.1

Compare Source

v0.176.0

Compare Source

v0.175.1

Compare Source

v0.175.0

Compare Source

v0.174.1

Compare Source

v0.174.0

Compare Source

v0.173.0

Compare Source

v0.172.0

Compare Source

v0.171.0

Compare Source

v0.170.0

Compare Source

v0.169.0

Compare Source

v0.168.0

Compare Source

v0.167.1

Compare Source

v0.167.0

Compare Source

v0.166.1

Compare Source

v0.166.0

Compare Source

v0.165.1

Compare Source

v0.165.0

Compare Source

v0.164.0

Compare Source

v0.163.0

Compare Source

v0.162.1

Compare Source

v0.162.0

Compare Source

v0.161.0

Compare Source

v0.160.2

Compare Source

[v0.160.1](https://redirect.github.com/flowtype/flow-bin/compare/dfdba3226b0db92254a6c11c96f2ea74b59883e7...01c86686f235cbf64


Configuration

📅 Schedule: Branch creation - "every weekend" in timezone US/Eastern, Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Copy link
Contributor Author

renovate bot commented May 5, 2024

⚠ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: yarn.lock
/opt/containerbase/tools/corepack/0.28.0/15.14.0/node_modules/corepack/dist/lib/corepack.cjs:9398
          if (typeof this !== "object" || this === null || !(#target in this)) {
                                                             ^^^^^^^

SyntaxError: Unexpected identifier
    at Object.compileFunction (node:vm:355:18)
    at wrapSafe (node:internal/modules/cjs/loader:1022:15)
    at Module._compile (node:internal/modules/cjs/loader:1056:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1121:10)
    at Module.load (node:internal/modules/cjs/loader:972:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Module.require (node:internal/modules/cjs/loader:996:19)
    at require (node:internal/modules/cjs/helpers:92:18)
    at Object.<anonymous> (/opt/containerbase/tools/corepack/0.28.0/15.14.0/node_modules/corepack/dist/yarn.js:3:1)
    at Module._compile (node:internal/modules/cjs/loader:1092:14)

@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 5, 2024 02:24 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 2f436ff to c77697c Compare May 6, 2024 13:22
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 6, 2024 13:24 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from c77697c to 57eae8b Compare May 6, 2024 16:18
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 6, 2024 16:18 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 57eae8b to 1eabfdf Compare May 6, 2024 18:30
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 6, 2024 18:31 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from eb2a622 to c69bff5 Compare May 6, 2024 20:31
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 6, 2024 20:32 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from c69bff5 to d15728e Compare May 7, 2024 09:55
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 7, 2024 09:57 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from d15728e to 46efabb Compare May 9, 2024 22:36
@renovate renovate bot changed the title fix(deps): update dependency flow-bin to ^0.235.0 fix(deps): update dependency flow-bin to ^0.236.0 May 9, 2024
@odlbot odlbot had a problem deploying to xpro-ci-pr-2979 May 9, 2024 22:39 Failure
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 46efabb to c9f38d6 Compare May 14, 2024 13:29
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from c9f38d6 to d9713f8 Compare May 16, 2024 15:40
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from d9713f8 to 573bfa7 Compare May 17, 2024 11:17
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 573bfa7 to 88c3f1c Compare May 17, 2024 12:26
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 88c3f1c to ba0e268 Compare May 17, 2024 12:36
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from ba0e268 to 1428841 Compare May 18, 2024 11:56
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 1428841 to e788cf8 Compare May 18, 2024 15:09
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch 6 times, most recently from 99e7160 to 323f20d Compare April 16, 2025 16:46
@renovate renovate bot changed the title fix(deps): update dependency flow-bin to ^0.267.0 fix(deps): update dependency flow-bin to ^0.268.0 Apr 16, 2025
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch 5 times, most recently from 2045dc5 to 54ad64f Compare April 25, 2025 07:31
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 54ad64f to 8b50e33 Compare April 28, 2025 15:37
@renovate renovate bot force-pushed the renovate/flow-bin-0.x branch from 8b50e33 to 6f7feff Compare April 29, 2025 00:49
@renovate renovate bot changed the title fix(deps): update dependency flow-bin to ^0.268.0 fix(deps): update dependency flow-bin to ^0.269.0 Apr 29, 2025
@asajjad2 asajjad2 self-assigned this Apr 29, 2025
Copy link
Contributor Author

renovate bot commented Apr 29, 2025

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@asajjad2
Copy link
Contributor

asajjad2 commented Apr 30, 2025

We had added esproposal.class_static_fields=enable and esproposal.class_instance_fields=enable to ignore ES class fields syntax. These configs were deprecated and are enabled by default as mentioned here.

suppress_comment config option was also deprecated and defaults to $FlowFixMe which works because our config was the same suppress_comment= \\(.\\|\n\\)*\\$FlowFixMe

Also encountered a [libdef-override] error because our custom module declaration for webpack in declarations.js conflicted with Flow's built-in definition. This was fixed by adding libdef-override=off to the [lints] section of our .flowconfig file, which specifically disables this lint check while maintaining all other type checking. Since this was the only instance of this error in our codebase, this approach was safe and straightforward.

@asajjad2
Copy link
Contributor

asajjad2 commented May 5, 2025

In one of the previous PRs, flow was incorrectly configured to just check .cjs files. When we remove that config so it checks all .js, .jsx, .mjs, .cjs and .json files, combined with this update - we get a lot of errors. Some are simple changes from bool -> boolean, but there are other various syntax updates which we would need to make.

Another problem is that for many 3rd party modules, we don't have library definition files. From what I understand, we can run flow-typed install to generate libdefs for them. Doing this will add pre-existing libdefs for as many libraries as it can find. However, there are many libraries for which these libdefs don't exist - flow-typed marks their exports as any, but I'm not sure if we want that. The other approach would be to manually create library definitions for these libraries.

@asajjad2
Copy link
Contributor

asajjad2 commented May 5, 2025

@arslanashraf7 should we go with flow-typed generated libdefs or stick with only checking .cjs files?

@Anas12091101
Copy link
Contributor

Anas12091101 commented Jun 4, 2025

@arslanashraf7 Should we consider transitioning our JSX files to TypeScript for static type checking, similar to MIT Learn? TypeScript has become the industry standard for type safety, and edX MFEs are also moving in that direction.
We can then remove this flow dependency

@arslanashraf7
Copy link
Contributor

@arslanashraf7 Should we consider transitioning our JSX files to TypeScript for static type checking, similar to MIT Learn? TypeScript has become the industry standard for type safety, and edX MFEs are also moving in that direction. We can then remove this flow dependency

I think that is a broader decision to make. We need to check if this is worth the effort since Learn is going to be the main platform moving forward. Even if we decide to transition from jsx to tsx, we should do that in a pre-planned separate ticket.

cc: @pdpinch @cachob

@arslanashraf7
Copy link
Contributor

Marking it blocked based on #2979 (comment)

@arslanashraf7
Copy link
Contributor

Marking it blocked based on #2979 (comment) + I think it is not worth doing this since we will be moving away from this app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants