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
Sublime dev #38383
Sublime dev #38383
Conversation
Quoting myself from the original PR:
|
@jtojnar done. |
Great, thanks. Although I still do not like the argument duplication very much. Could you do something like, for example, Firefox does? |
@jtojnar done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, cc maintainers @wmertens, @demin-dmitriy, @zimbatm
x32sha256 = "0dgpx4wij2m77f478p746qadavab172166bghxmj7fb61nvw9v5i"; | ||
x64sha256 = "06b554d2cvpxc976rvh89ix3kqc7klnngvk070xrs8wbyb221qcw"; | ||
} {}; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the rec
keywords in this file don't seem to be necessary. rec
is useful to re-export keys of the current attrset as similarly-named variables into the local scope.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done: 45e72ed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good overall, no objections for a merge
@MasseGuillaume The sublime(2) attribute was removed by this PR. I suppose this was by accident. |
Right, sorry, fixed in 8e18b7c |
Resurecting: #26718