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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

switch PyThaiNLP source code to SPDX license ID #876

Merged
merged 4 commits into from
Dec 4, 2023

Conversation

pavaris-pm
Copy link
Contributor

@pavaris-pm pavaris-pm commented Dec 3, 2023

What does this changes

According to issue #874 , in this PR, i already switch the license to SPDX already by replacing it with the new one as mentioned in that issue krub. Tag @bact since i have made changes and you are an issue owner. You can take a review at it and inform me if i miss something in the source code krub.

Fix #874

Your checklist for this pull request

馃毃Please review the guidelines for contributing to this repository.

  • Passed code styles and structures
  • Passed code linting checks and unit test

Comment on lines 1 to 3
# -*- coding: utf-8 -*-
# PyThaiNLP: Thai Natural Language Processing in Python
#
# Copyright (C) 2016-2023 PyThaiNLP Project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
# SPDX-FileCopyrightText: Copyright (C) 2016-2023 PyThaiNLP Project.
# SPDX-License-Identifier: Apache-2.0
Copy link
Contributor Author

@pavaris-pm pavaris-pm Dec 3, 2023

Choose a reason for hiding this comment

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

@bact here is an example of the change i've made according to your issue krub.

@coveralls
Copy link

coveralls commented Dec 3, 2023

Coverage Status

coverage: 86.133% (+0.03%) from 86.104%
when pulling f86c23b on pavaris-pm:update-license
into f877567 on PyThaiNLP:dev.

@bact
Copy link
Member

bact commented Dec 3, 2023

Thank a lot @pavaris-pm . This is so fast.

Do you mind to changes two things please?:

  1. Using this copyright text line instead:
SPDX-FileCopyrightText: Copyright 2016-2023 PyThaiNLP Project

(I decided to follow template in SPDX website)

  1. Also replace the license text in setup.py

Thank you.

@bact bact added the documentation improve documentation and test cases label Dec 3, 2023
@bact bact added this to the 5.0 milestone Dec 3, 2023
@bact bact added this to In progress in PyThaiNLP Dec 3, 2023
@pavaris-pm
Copy link
Contributor Author

Thank a lot @pavaris-pm . This is so fast.

Do you mind to changes two things please?:

  1. Using this copyright text line instead:

SPDX-FileCopyrightText: Copyright 2016-2023 PyThaiNLP Project

(I decided to follow template in SPDX website)

  1. Also replace the license text in setup.py

Thank you.

Roger that, I will add it accordingly within today krub.

Copy link

sonarcloud bot commented Dec 4, 2023

Kudos, SonarCloud Quality Gate passed!聽 聽 Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@pavaris-pm
Copy link
Contributor Author

@bact i already add and update according to your review. please review my latest push krub.

@bact bact self-requested a review December 4, 2023 09:03
Copy link
Member

@bact bact left a comment

Choose a reason for hiding this comment

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

Thank you :)

@bact bact merged commit d213a8a into PyThaiNLP:dev Dec 4, 2023
10 of 14 checks passed
PyThaiNLP automation moved this from In progress to Done Dec 4, 2023
@bact bact mentioned this pull request Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation improve documentation and test cases
Projects
PyThaiNLP
  
Done
Development

Successfully merging this pull request may close these issues.

Switching to SPDX license identifiers in PyThaiNLP sources
3 participants