Skip to content

don't put the license on these files #909

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 16, 2024
Merged

don't put the license on these files #909

merged 1 commit into from
Jul 16, 2024

Conversation

byjlw
Copy link
Contributor

@byjlw byjlw commented Jul 16, 2024

No description provided.

@byjlw byjlw requested review from malfet and Jack-Khuu July 16, 2024 20:14
@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Jul 16, 2024 •
edited
Loading

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/torchchat/909

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 4abf4ad with merge base 4ca7f49 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 16, 2024
@byjlw byjlw merged commit 22d7529 into main Jul 16, 2024
51 checks passed
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
f304665 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
93199e8 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
don't put the license on these files (#909)
68e2f20
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
139d2ac 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
621d548 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
4d1f6d5 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
a70f4ba 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
453c16a 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
vmpuri pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @vmpuri
don't put the license on these files (#909)
1b75325
malfet pushed a commit that referenced this pull request Jul 17, 2024
@byjlw @malfet
need license header on every file (#904)
6372e36 
Add license header to files (#905)

* need license header on every file

* apparently the validator only reports the first violation it finds not all of them

* typeo

don't put the license on these files (#909)

add license to source files (#910)

this should be the final one (#911)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jack-Khuu Jack-Khuu Jack-Khuu approved these changes

@malfet malfet Awaiting requested review from malfet

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@byjlw @Jack-Khuu @facebook-github-bot