Skip to content
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

fix(ses): Consistently name console methods #2654

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

gibson042
Copy link
Contributor

Ref #2643
Ref 5e8fd33

Description

Unify the inconsistent approaches to creating [virtual] console functions with appropriate "name" properties.

Security Considerations

None.

Scaling Considerations

n/a

Documentation Considerations

n/a

Testing Considerations

I don't think these details are covered by testing, but it seems like an acceptable gap for now.

Compatibility Considerations

None.

Upgrade Considerations

None.

@gibson042 gibson042 requested a review from kriskowal December 5, 2024 19:21
Copy link
Member

@kriskowal kriskowal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a refactor to me. Consider refactor(ses): for the commit message. I don’t see it, but if you did incidentally fix a missing .name, feat or fix seem okay, but it would be neat to have a test.

@gibson042 gibson042 force-pushed the gibson-2024-12-console-method-names branch from 38accce to fa7a1c4 Compare December 6, 2024 19:34
@gibson042 gibson042 changed the title feat(ses): Consistently name console methods fix(ses): Consistently name console methods Dec 6, 2024
@gibson042 gibson042 merged commit 4216217 into master Dec 6, 2024
15 checks passed
@gibson042 gibson042 deleted the gibson-2024-12-console-method-names branch December 6, 2024 19:58
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.

2 participants