ysoftwareab / eslint-plugin-y

Fork of https://github.com/tobiipro/eslint-config-firecloud
The Unlicense
1 stars 0 forks source link

breaking: prefer single-var destructuring without multiline #41

Closed andreineculau closed 4 years ago

andreineculau commented 4 years ago
text dump at https://raw.githubusercontent.com/rokmoln/zz-issues/master/tobiipro/eslint-config-firecloud/41.txt
breaking: prefer single-var destructuring without multiline #41

   allow-singleline-singlevar-destruct

   master allow-singleline-singlevar-destruct
   @andreineculau
   [45]andreineculau opened this pull request
   5 months ago

   Labels
   [46]enhancement
     * Fixes:
     * Breaking change: [x]
     __________________________________________________________________

   had a sparse look at git diffs and it is not that common to have turn a
   single-var destructuring into multi-var destructuring, which was the
   main argument (smaller diffs) for requiring multiline even for
   single-var destructuring

   [47]andreineculau added a commit [48]5 months ago
   [49]@andreineculau [50]breaking: prefer single-var destructuring
   without multiline

   andreineculau added the [51]enhancement label [52]5 months ago

   andreineculau requested a review from tobiiasl [53]5 months ago

   andreineculau added this to In progress in [54]Public (Open Source) via
   automation [55]5 months ago

   @tobiiasl
   [56]tobiiasl approved these changes [57]5 months ago

   andreineculau added this to the breaking milestone [58]5 months ago

   andreineculau referenced this pull request from commit [59]e1af02e
   [60]4 months ago

   andreineculau merged commit e1af02e into master [61]4 months ago

   Public (Open Source) automation moved this from In progress to Done
   [62]4 months ago

   andreineculau deleted the allow-singleline-singlevar-destruct branch
   [63]4 months ago

   ankitmth removed this from Done in [64]Public (Open Source) [65]17 days
   ago