Skip to content

Refactor and update DictSQLite package and GitHub Actions Workflows - #1

Closed
yuito-it wants to merge 6 commits into
disnana:mainfrom
yuito-it:main
Closed

yuito-it wants to merge 6 commits into
disnana:mainfrom
yuito-it:main

Conversation

@yuito-it

Copy link
Copy Markdown
Member

This pull request includes several commits that refactor and update the DictSQLite package. The commits include changes to the package's README and CONTRIBUTORS files, the addition of new files for linting and publishing, and updates to the package's configuration and dependencies. The changes aim to improve the functionality and maintainability of the package.

@yuito-it

yuito-it commented Aug 30, 2024 •

Copy link
Copy Markdown
Member Author

今回のプルリクは、

  • ソースコードがない問題を修正
  • メールではなくIssueにFBを投げてもらうように修正
  • PylintのWorkflow化
  • GitHubのRelease時に自動的にPyPlに公開するようにする
    というものです。

@yuito-it

Copy link
Copy Markdown
Member Author

PyPlに限らず、パッケージを公開するということは基本的にはオープンソース化するということなので、ソースコードの公開は基本的にマストです。
その上で、いかに作業を効率化できるかと考えると、やはりGitHubActionsではないでしょうかね...

@yuito-it

yuito-it commented Aug 30, 2024 •

Copy link
Copy Markdown
Member Author

Pylintが通っているかどうかでそのバージョンを使うかどうかを見分けることも多々あります。
(とドワンゴの本職が申しておりました)

@yuito-it

Copy link
Copy Markdown
Member Author

Pylintはあれですね。
Pythonのコード内にエラーがないかとかをチェックしてくれるツールです。

@yuito-it

yuito-it commented Aug 30, 2024 •

Copy link
Copy Markdown
Member Author

こちらにソースコードを追加しました。

普通はsetup.pyとかもですね。
基本的に、setup.pyをビルドした際に出る、配布ファイルは含みませんので、.gitignoreに追加しておけば良いと思います。

@yuito-it

Copy link
Copy Markdown
Member Author

とりあえず、このPRは使わないようなのでクローズします...

@yuito-it yuito-it closed this Aug 30, 2024
Copilot AI added a commit that referenced this pull request Oct 6, 2025
Co-authored-by: harumaki4649 <83683593+harumaki4649@users.noreply.github.com>
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.

1 participant