Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Re-order actions you can take with a Result#201

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
illicitonion wants to merge1 commit intomainmatter:main
base:main
Choose a base branch
Loading
fromillicitonion:reorder-result-outcomes

Conversation

@illicitonion
Copy link
Contributor

It's generally better practice to handle or propagate errors, rather than panicking in response to them. This edit moves panicking to be thelast option introduced, rather than the first. It also adds caveats to avoid doing so, and explicitly mentions propagating as something to consider.

It's generally better practice to handle or propagate errors, ratherthan panicking in response to them. This edit moves panicking to be the_last_ option introduced, rather than the first. It also adds caveats toavoid doing so, and explicitly mentions propagating as something toconsider.
@netlify
Copy link

netlifybot commentedOct 31, 2024
edited
Loading

Deploy Preview fortaupe-lily-3539f6 ready!

NameLink
🔨 Latest commit2e2f6b9
🔍 Latest deploy loghttps://app.netlify.com/sites/taupe-lily-3539f6/deploys/67238ad3f2757d0008ee3617
😎 Deploy Previewhttps://deploy-preview-201--taupe-lily-3539f6.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify site configuration.

@hdoordt
Copy link
Member

@illicitonion Thanks for your contribution! I think it's a good addition overall. However, the story about the?-operator is a bit more complicated, as it will do a conversion as well if possible and necessary. Can you update your text so that it covers that part as well?

@LukeMathWalker
Copy link
Collaborator

We should be careful here with respect to when concepts are introduced—we have no knowledge of conversion traits at this stage, so we can't give a precise explanation as to how? actually behaves.

My suggestion is to remove? from this bullet list, make it its own sub-section in the same chapter and clarify in a footnote that there's more to this that'll be discussed later, with a link to the chapter where we do. That chapter's text also needs to be adjusted to reflect the fact that? has already been introduced.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@illicitonion@hdoordt@LukeMathWalker

[8]ページ先頭

©2009-2025 Movatter.jp