Ecosyste.ms: Timeline

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

lrytz

lrytz pushed 1 commit to annot-overload lrytz/scala
  • AnnotationInfo.overload represents the used annotation constructor Pickling will be tricky though, cannot pickle / u... 1b7765c

View on GitHub

lrytz created a branch on lrytz/scala

annot-overload - The Scala programming language

lrytz created a comment on an issue on scala/scala-dev
Another item to fix - though it will be less relevant after the other fixes. For an annotation with auxiliary constructors, the `AnnotationInfo` doesn't tell which overload was selected. Probably j...

View on GitHub

lrytz created a comment on a pull request on sbt/zinc
> Not sure why Scala CLA is still failing Seems the checker doesn't follow redirects. https://github.com/sbt/zinc/pull/1513

View on GitHub

lrytz opened a pull request on sbt/zinc
new CLA url
lrytz created a comment on a pull request on scala/scala
Thanks a lot for the sharp-eyed (and -brained) review @NthPortal!

View on GitHub

lrytz created a review comment on a pull request on scala/scala3
`compiletime.ops.S` is intrinsic right? Maybe we could alter symbol / type somehow (in definitions?) to make `isMatchAlias` true?

View on GitHub

lrytz created a review on a pull request on scala/scala3
Looks very good!

View on GitHub

lrytz pushed 1 commit to t13062 lrytz/scala
  • second round of review feedback f462b47

View on GitHub

lrytz pushed 2 commits to t13062 lrytz/scala
  • Ensure LazyList.initState assigns evaluated head / tail Also simplify a couple of operation implementations and alig... ec8b7b4
  • second round of review feedback 16daf49

View on GitHub

lrytz created a review comment on a pull request on scala/scala
I'm going for `eagerHeadOperation`, but leaving `eagerCons` as is.

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
I'm pro comma, for the cycle size I have no opinion :)

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
I guess because its parameters are unrelated to LazyList, though the return type needs to work out, so we'd catch it if it picked up some other method. I also left the prefix for `LazyList.from` / ...

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
I have no preference either way...

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
I could change it to `if (evaluated eq Empty)`

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
No change here, as far as I can tell. Before, `lazyState` returning a `State` was enforcing evaluation within the `lazyState` function. Now we do it here at the callsite. It was important for cycle...

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
ups, that slipped in from reviewing https://github.com/scala/scala/pull/10766

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a review comment on a pull request on scala/scala
> > it prevents assigning non-evaluated fields to `_head` / `_state` in `initState`, which would be a bug > > to clarify, would it be a bug because it would cause multiple evaluation? You're ...

View on GitHub

lrytz created a review on a pull request on scala/scala

View on GitHub

lrytz created a comment on a pull request on sbt/zinc
> Getting a 500 error trying to sign the Scala CLA I reported it

View on GitHub

lrytz created a review on a pull request on scala/scala
If we're keeping all 3 methods around I'd like the docs to say when to use which - I don't know at this point. Is there a big risk that existing custom maps would violate the new iteration order...

View on GitHub

lrytz created a review on a pull request on scala/scala
If we're keeping all 3 methods around I'd like the docs to say when to use which - I don't know at this point. Is there a big risk that existing custom maps would violate the new iteration order...

View on GitHub

Load more