Skip to content

Conversation

@youngbeom-shin
Copy link
Member

What this PR does:

  • Removed negative margin class from div
  • Cleaned up layout for better alignment
    Which issue(s) this PR fixes:

Fixes # gitlab-1034

Type of changes
Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Feature/Issue validation/testing:

Please describe the tests that you ran to verify your changes and relevant result summary.

  • Test A

  • Test B

  • Logs

Special notes for your reviewer:

Checklist:

  • I have added unit/e2e tests that prove your fix is effective or that this feature works.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have reviewed my own code and ensured that it follows the project's style guidelines.

Release note:


@youngbeom-shin youngbeom-shin self-assigned this Aug 1, 2025
@youngbeom-shin youngbeom-shin added the GitLab linked linked to a GitLab task or issue. label Aug 1, 2025
Copy link
Collaborator

@zhendi zhendi left a comment

Choose a reason for hiding this comment

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

LGTM

@zhendi zhendi merged commit 53d1362 into main Aug 1, 2025
3 checks passed
@zhendi zhendi deleted the csghub__remove-unnecessary-margin-class-in-finetune branch August 1, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GitLab linked linked to a GitLab task or issue. local-verified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants