Skip to content

Lwt_unix.yield was deprecated in favor of Lwt.pause#297

Merged
avsm merged 1 commit intoocurrent:masterfrom
MisterDA:Lwt_main.pause-deprecated-for-Lwt.pause
Oct 26, 2021
Merged

Lwt_unix.yield was deprecated in favor of Lwt.pause#297
avsm merged 1 commit intoocurrent:masterfrom
MisterDA:Lwt_main.pause-deprecated-for-Lwt.pause

Conversation

@MisterDA
Copy link
Copy Markdown
Contributor

@MisterDA MisterDA commented Oct 4, 2021

Deprecated in the lwt dev, but it's a safe change to make now.

Deprecated in the lwt dev, but it's a safe change to make now.
@kit-ty-kate
Copy link
Copy Markdown
Contributor

kit-ty-kate commented Oct 4, 2021

when was this function added? I suspect a lower bound bump might be needed

@dra27
Copy link
Copy Markdown
Contributor

dra27 commented Oct 4, 2021

It's at least in Lwt 3.0.0 and ocurrent already requires >= 4.3.0

@MisterDA
Copy link
Copy Markdown
Contributor Author

MisterDA commented Oct 4, 2021

ocsigen/lwt@f3c5fdb before 2.1.0.

@avsm avsm merged commit 7345aec into ocurrent:master Oct 26, 2021
@MisterDA MisterDA deleted the Lwt_main.pause-deprecated-for-Lwt.pause branch October 26, 2021 16:16
tmcgilchrist added a commit to tmcgilchrist/opam-repository that referenced this pull request Apr 7, 2022
…, current_github, current_git, current_examples, current_docker and current (0.6)

CHANGES:

Core:

- Implement labelling of clusters on the Graphviz diagram
  (@ewanmellor ocurrent/ocurrent#255)

- Abort propagation on constant/equal changes (@art-w ocurrent/ocurrent#318)

API:

- GitHub: Record build status using CheckRun (@tmcgilchrist ocurrent/ocurrent#279)

- GitHub: Add details_url to check_run. (@tmcgilchrist ocurrent/ocurrent#282)

- GitHub: Add Current_github.Api.cmdliner_opt to allow writing
  pipelines which can optionally be run as GitHub apps. (@talex5 ocurrent/ocurrent#281)

- GitHub: Provide markdown details for CheckRun. (@tmcgilchrist ocurrent/ocurrent#288)

- GitHub: Fix wrong name used for repository (@tmcgilchrist ocurrent/ocurrent#289 ocurrent/ocurrent#290)

- GitHub: Support Github rebuild via webooks. (@tmcgilchrist ocurrent/ocurrent#283)

- GitHub: monitor GraphQL queries (@art-w ocurrent/ocurrent#298)

- GitHub: Limit CheckRunStatus summary and text fields to 65535.
  (@tmcgilchrist ocurrent/ocurrent#300)

- GitHub: Log extra context for Webhook validation failure.
  (@tmcgilchrist ocurrent/ocurrent#302)

- GitLab: Initial GitLab plugin work. (@tmcgilchrist ocurrent/ocurrent#299)

- Git: Make git reset less verbose (@kit-ty-kate ocurrent/ocurrent#293)

Web UI:

- Use Lwt.pause instead of Lwt_unix.yield (@MisterDA ocurrent/ocurrent#297)

- Use `ansi` instead of `current_ansi` (@samoht ocurrent/ocurrent#321)

- Show line numbers and allow jumping to specific lines in job
  logs (@punchagan ocurrent/ocurrent#309)

Docker:

- Explicitly set confirmation levels to allow for
  manually triggered jobs. (@tmcgilchrist ocurrent/ocurrent#304)

- Stop using `Dockerfile.t` completely and use strings instead.
  (@MisterDA ocurrent/ocurrent#301 ocurrent/ocurrent#316)

Other:

- Update to cohttp 4.0.0 (ocurrent/ocurrent#274, @talex5)

- Move `Current_incr` to its own repository (ocurrent/ocurrent#284, @talex5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants