Unintentionally global variables stat and ret in async_lib?
まだ誰も着手していません。
評価
調査の方向性
Open lua/plenary/async_lib/util.lua around lines 292-302 and inspect how stat and ret are assigned and used. The issue is done when those names no longer unintentionally create or modify globals while preserving the existing async_lib behavior.
索引モデルが issue の本文から書いたものです。
説明
It looks like stat and ret should be local in this code and not global.
This is a common mistake in lua that happens easily I guess. Found by lua language server autocomplete.
To make them local would make them encapsulated, and you avoid risk - people using the unintentionally used variables, or overwriting people's globals.
Link:
- 主要言語
- Lua
- スター
- 3.5k
- フォーク
- 341
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nvim-lua/plenary.nvim のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
nvim-lua/plenary.nvim#682 ·
-
uniform usage of `os.getenv`対応中かも @rakotomandimby が 124 日前に担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
nvim-lua/plenary.nvim#680 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
nvim-lua/plenary.nvim#675 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 28/100
nvim-lua/plenary.nvim#672 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
nvim-lua/plenary.nvim#671 · コメント 1 件 ·
nvim-lua/plenary.nvim の issue をすべて見る
似ている issue
-
bug good first issue ready ticket
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Domktorymysli/vclu#20 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
InauguralSystems/EigenLua#60 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 1/5 1〜3時間 初心者へのやさしさ 85/100
bradcypert/plum#265 ·
メンテナーはふだん 1 日以内に返信