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

Lexographically smallest after a swap#99

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

Merged

Conversation

@meetmukul1993
Copy link
Contributor

@meetmukul1993meetmukul1993 commentedOct 17, 2024•
edited
Loading

Title of the problem

3216. Lexicographically Smallest String After a Swap

Checklist before requesting a review

  • I have checked theguide for contributions.
  • I have added the description at the top of the solution file.
  • I have added a test file for the problem.
  • I have added an entry to theREADME file with a link to the new file.
  • I have run linter (npx eslint LeetcodeProblems LeetcodeProblemsTests) and unit tests (node Test.js) and they pass.More info.

Check theguide andthis PR example.

Copy link
Owner

@ignacio-chiazzoignacio-chiazzo left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Owner

@ignacio-chiazzoignacio-chiazzo left a comment

Choose a reason for hiding this comment

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

Could you fix the linter issues by runningnpx eslint LeetcodeProblems LeetcodeProblemsTests

@meetmukul1993
Copy link
ContributorAuthor

@ignacio-chiazzo updated the PR, can u please have a look again. Thanks

Copy link
Owner

@ignacio-chiazzoignacio-chiazzo left a comment

Choose a reason for hiding this comment

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

Thanks

@meetmukul1993
Copy link
ContributorAuthor

@ignacio-chiazzo Seems like the lint check runs after a maintainer approves it, earlier it failed, have updated the code again. can u please approve the lint execution

@ignacio-chiazzoignacio-chiazzo merged commit03b48f6 intoignacio-chiazzo:masterOct 19, 2024
1 check passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ignacio-chiazzoignacio-chiazzoignacio-chiazzo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@meetmukul1993@ignacio-chiazzo
[8]ページ先頭
©2009-2025 Movatter.jp