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

feat: improve side label UI #886

Merged

Conversation

tyn1998
Copy link
Member

@tyn1998 tyn1998 commented Oct 8, 2024

Brief Information

This pull request is in the type of (more info about types):

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • test

Related issues (all available keywords):

Details

image

2024-10-08.22.33.31.mov

Checklist

Others

@tyn1998 tyn1998 requested a review from frank-zsy October 8, 2024 14:17
@frank-zsy
Copy link
Contributor

Looks great, what will this be like for https://github.com/ray-project/ray ?

@tyn1998
Copy link
Member Author

tyn1998 commented Oct 9, 2024

Looks great, what will this be like for https://github.com/ray-project/ray ?

FYI,

image

@frank-zsy
Copy link
Contributor

frank-zsy commented Oct 9, 2024

Sorry, what I mean is actually https://github.com/langchain-ai/langchain , because this repo does not have any GitHub tags.

To append labels after GitHub tags, we should careful about the repos without any tags.

@tyn1998
Copy link
Member Author

tyn1998 commented Oct 10, 2024

https://github.com/langchain-ai/langchain

I didn't consider that scenario. I will do a quick fix on this issue.

@tyn1998
Copy link
Member Author

tyn1998 commented Oct 10, 2024

image

Now it should work in repos without any tag :)

@frank-zsy
Copy link
Contributor

@wangyantong2000 Could you have a look on this.

@wangyantong2000
Copy link
Collaborator

@wangyantong2000 Could you have a look on this.

OK

@wangyantong2000
Copy link
Collaborator

I tried several repositories and the previous issues were resolved.

@wangyantong2000 wangyantong2000 merged commit 2d6e90b into hypertrons:master Oct 12, 2024
2 checks passed
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.

[Refactor] Optimize the UI of OpenDigger labels
3 participants