Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert typescript CI Lint changes #6602

Merged
merged 2 commits into from
May 8, 2023

Conversation

somebody1234
Copy link
Contributor

Pull Request Description

Hotfix revert to make Lint CI work again

Important Notes

None

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
  • All code follows the
    Scala,
    Java,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • All code has been tested:
    • Unit tests have been written where possible.
    • If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@somebody1234 somebody1234 added the CI: No changelog needed Do not require a changelog entry for this PR. label May 8, 2023
@somebody1234 somebody1234 changed the title Revert typescript CI Lint changes Revert TypeScript CI Lint changes May 8, 2023
@somebody1234
Copy link
Contributor Author

(proper TypeScript CI lint coming soon)

@somebody1234
Copy link
Contributor Author

@mwu-tow could you perhaps review this?

@somebody1234
Copy link
Contributor Author

well actually... the fix is ready so...

@somebody1234 somebody1234 force-pushed the wip/sb/revert-typescript-lint-changes branch from b5e5ad8 to afbe58a Compare May 8, 2023 15:47
@somebody1234 somebody1234 changed the title Revert TypeScript CI Lint changes Properly fix TypeScript CI Lint May 8, 2023
@somebody1234
Copy link
Contributor Author

feel free to split this into two PRs, but given that the fix should be ready, it should be faster to fix it as well?

@somebody1234
Copy link
Contributor Author

somebody1234 commented May 8, 2023

darn, lint CI broke: https://github.com/enso-org/enso/actions/runs/4916999193/jobs/8781496456?pr=6602

GitHub
Hybrid visual and textual functional programming. Contribute to enso-org/enso development by creating an account on GitHub.

@somebody1234 somebody1234 force-pushed the wip/sb/revert-typescript-lint-changes branch from afbe58a to b315b8d Compare May 8, 2023 15:56
@somebody1234
Copy link
Contributor Author

yeah... this is not working :( splitting the PR up

@somebody1234 somebody1234 changed the title Properly fix TypeScript CI Lint Revert typescript CI Lint changes May 8, 2023
@somebody1234 somebody1234 added the CI: Ready to merge This PR is eligible for automatic merge label May 8, 2023
@somebody1234
Copy link
Contributor Author

adding "ready to merge" label as:

  • it's a hotfix
  • it's a very small change
  • it's a revert
    so basically, urgent + unlikely to break anything

@mwu-tow mwu-tow merged commit 0540430 into develop May 8, 2023
@mwu-tow mwu-tow deleted the wip/sb/revert-typescript-lint-changes branch May 8, 2023 17:09
Procrat added a commit that referenced this pull request May 10, 2023
* develop: (28 commits)
  Add tests for Date.until, Date.next and Date.previous. (#6606)
  Improve `Non_Unique_Primary_Key` error, split file format detection into read/write, improve SQLite format detection (#6604)
  tokenize_to_columns or parse_to_columns results in a single column we shouldn't add the  1 (#6607)
  Fix node editing race condition (#6594)
  Add format to the in-memory Column (#6538)
  Fix dashboard issues (part 2) (#6511)
  Fix visualisation type selector artifacts rendered after node preview visualisation was closed. (#6575)
  Revert typescript CI Lint changes (#6602)
  Fix the Engine version check in GUI (#6570)
  Show error pop-up when failing to rename a project (#6366)
  Small changes from Book Club issues (#6533)
  "at_least_one" flag for tokenize_to_rows (#6539)
  Benchmark Engine job runs only engine, not Enso benchmarks (#6534)
  Catch 5813 and avoid crash (#6585)
  Fix opening links in desktop IDE (#6507)
  Identify SyntaxError exception and avoid printing a stack trace (#6574)
  Fix dashboard issues (#6502)
  Let ChangesetBuilder.invalidated search even container elements (#6548)
  Fix #5075: stop panning on full-screen visualisation (#6530)
  Only `Join_Kind.Inner` removes the common-named columns (#6564)
  ...
Procrat added a commit that referenced this pull request May 10, 2023
…ing-6287

* develop:
  Fix issues with missing sourcemaps (#6572)
  Fix asset delete; implement project delete and project rename (#6566)
  Fix #6377: Change ctrl-r shortcut (#6620)
  Add tests for Date.until, Date.next and Date.previous. (#6606)
  Improve `Non_Unique_Primary_Key` error, split file format detection into read/write, improve SQLite format detection (#6604)
  tokenize_to_columns or parse_to_columns results in a single column we shouldn't add the  1 (#6607)
  Fix node editing race condition (#6594)
  Add format to the in-memory Column (#6538)
  Fix dashboard issues (part 2) (#6511)
  Fix visualisation type selector artifacts rendered after node preview visualisation was closed. (#6575)
  Revert typescript CI Lint changes (#6602)
  Fix the Engine version check in GUI (#6570)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants