Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

erights

erights created a review comment on a pull request on endojs/endo
I believe this line is vestigial and should simply be deleted. ```suggestion ```

View on GitHub

erights created a review on a pull request on endojs/endo
LGTM, thanks!

View on GitHub

erights created a review on a pull request on endojs/endo
LGTM, thanks!

View on GitHub

erights created a review comment on a pull request on endojs/endo
Why do we no longer need the try/finally? Or did we already not need it?

View on GitHub

erights created a review on a pull request on endojs/endo
Review not finished, but wanted to get these in front of you first.

View on GitHub

erights created a review on a pull request on endojs/endo
Review not finished, but wanted to get these in front of you first.

View on GitHub

erights closed an issue on endojs/endo
Permit Promise.try
## What is the Problem Being Solved? Promise.try ships with Node.js 23 and is now Stage 4. SES should permit it. ## Description of the Design Add and allow in SES permits ## Security Co...
erights closed an issue on endojs/endo
Permit Promise.try
## What is the Problem Being Solved? Promise.try ships with Node.js 23 and is now Stage 4. SES should permit it. ## Description of the Design Add and allow in SES permits ## Security Co...
erights pushed 1 commit to master endojs/endo
  • feat(ses): permit Promise.prototype.try (#2609) Closes: #2607 ## Description > "There is no do. There is only... d37a434

View on GitHub

erights deleted a branch endojs/endo

markm-2607-permit-promise-try

erights closed a pull request on endojs/endo
feat(ses): permit Promise.prototype.try
Closes: #2607 ## Description > "There is no do. There is only try." > —Noda See also https://github.com/tc39/proposal-promise-try ### Security Considerations none ### Scaling Co...
erights pushed 3 commits to markm-2607-permit-promise-try endojs/endo
  • feat(compartment-mapper): add Compartment option to captureFromMap This just allows `captureFromMap()` to receive a ... 1ac2156
  • feat(compartment-mapper): add Compartment option to captureFromMap (#2612) This just allows `captureFromMap()` to re... 7c32c62
  • feat(ses): permit Promise.try a8e6a63

View on GitHub

erights created a comment on a pull request on endojs/endo
> You beat me to it, but `Promise.try` needs to be on the constructor and not the prototype. Doh! Fixed. PTAL

View on GitHub

erights pushed 1 commit to markm-2607-permit-promise-try endojs/endo
  • feat(ses): review suggestion 11c75d8

View on GitHub

erights pushed 1 commit to main Agoric/proposal-stabilize

View on GitHub

erights created a branch on Agoric/proposal-stabilize

main - Proposal for tc39 of new integrity "level" protecting against both override mistakes and proxy reentrancy

erights created a repository: Agoric/proposal-stabilize

https://github.com/Agoric/proposal-stabilize - Proposal for tc39 of new integrity "level" protecting against both override mistakes and proxy reentrancy

erights created a review on a pull request on tc39/proposal-shadowrealm
I agree with @mhofman 's comments in this PR's conversations.

View on GitHub

erights pushed 1 commit to markm-2607-permit-promise-try endojs/endo
  • feat(ses): permit Promise.try daba7f9

View on GitHub

erights pushed 1 commit to markm-2607-permit-promise-try endojs/endo
  • feat(ses): permit Promise.try 354a325

View on GitHub

erights opened a pull request on endojs/endo
feat(ses): permit Promise.prototype.try
Closes: #2607 Refs: #XXXX ## Description "There is no do. There is only try." --Anti-Yoda See also https://github.com/tc39/proposal-promise-try ### Security Considerations ...
erights created a comment on an issue on endojs/endo
See https://github.com/tc39/proposal-promise-try

View on GitHub

erights created a branch on endojs/endo

markm-2607-permit-promise-try - Endo is a distributed secure JavaScript sandbox, based on SES

erights created a comment on a pull request on endojs/endo
> Do you have further comments or reason(s) to delay merging now that @kriskowal has approved? I have not found time to review this in enough depth to approve it. But I saw no red flags and am s...

View on GitHub

erights created a review on a pull request on endojs/endo
LGTM

View on GitHub

erights created a comment on a pull request on endojs/endo
Just skimming, but saw the q at end. rankCompare is a full order. I cannot answer NaN. (Unless this PR is making a more radical change than I thought. For all promises p,q, compareRank(p,q) === 0

View on GitHub

erights created a review on a pull request on Agoric/agoric-sdk
All my issues are dealt with well, so approving to remove my blockage. But you should still iterate with @mhofman to his satisfaction before merging.

View on GitHub

erights created a review comment on a pull request on Agoric/agoric-sdk
IIUC, looks good now. Thx.

View on GitHub

erights created a review on a pull request on Agoric/agoric-sdk

View on GitHub

erights created a review comment on a pull request on Agoric/agoric-sdk
Yes, ephemera would be correct.

View on GitHub

Load more