Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

リファクタリング&canvas使用コマンドをImages Featureに (#74) #96

Merged
merged 12 commits into from
May 2, 2024

Conversation

takejohn
Copy link
Member

@takejohn takejohn commented May 2, 2024

内容

#74 関連のリファクタリングです。
と思ったけど config.json で images Feature を切れる機能を追加したからリファクタリングじゃねぇや

変更点

  • core パッケージを作成して common, internal, util ディレクトリを core に移動
  • 未使用ファイルの webserver.js を削除
  • globalban.js, templinks.js で config を json でなくソースからインポート
  • canvas を使用する機能を別の Feature に移動して config で切れるように

チェックリスト:

  • このリポジトリのコードスタイルに沿っているか
  • 自分自身で動作確認を行ったか、また、それは正常に動作したか

@takejohn takejohn changed the title リファクタリング (#74) リファクタリング&canvas使用コマンドをImages Featureに (#74) May 2, 2024
@takejohn takejohn marked this pull request as ready for review May 2, 2024 13:27
@takejohn
Copy link
Member Author

takejohn commented May 2, 2024

Done!

@takejohn takejohn requested a review from ringo360 May 2, 2024 13:27
@ringo360
Copy link
Member

ringo360 commented May 2, 2024

🚀

@takejohn takejohn linked an issue May 2, 2024 that may be closed by this pull request
14 tasks
@takejohn takejohn removed a link to an issue May 2, 2024
14 tasks
@takejohn
Copy link
Member Author

takejohn commented May 2, 2024

#74 を close はしないな

Copy link
Member

@ringo360 ringo360 left a comment

Choose a reason for hiding this comment

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

Ready

@ringo360 ringo360 merged commit 4f1e4ec into v14-dev May 2, 2024
2 checks passed
@takejohn takejohn deleted the dev/74-refactoring branch May 2, 2024 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants