fix: eslint issues in sample apps #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arunvishnun
added a commit
that referenced
this pull request
Mar 21, 2023
arunvishnun
added a commit
that referenced
this pull request
Mar 22, 2023
arunvishnun
added a commit
that referenced
this pull request
Mar 24, 2023
arunvishnun
added a commit
that referenced
this pull request
Mar 24, 2023
arunvishnun
added a commit
that referenced
this pull request
Mar 29, 2023
…ode-io#1946) * [WIP] Use rush + pnpm for managing monorepo (electrode-io#1907) * Add rush config files * Add few projects for testing install and build Author: Ashu Verma <ashuverma@outlook.com> * Update Github Workflow to use rush * Add all the packages and fix dependencies Include All the packages in the repo into rush. Fix missing build scripts. Add missing dependencies. * Enabling tests. All of them still don't work * add node 16 in ci (electrode-io#1901) * add node 16 in ci * [fix]: upgrade xarc-opt-sass to pass node 16 build (electrode-io#1902) Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com> Co-authored-by: mristic505 <mateja.ristic@gmail.com> * Update github action and fix tests (electrode-io#1910) * Update github action and adding in several test fixes with rush Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> * feat: merge changes from master and fix poc-subapp (electrode-io#1919) * add two options for improving webpack performance (electrode-io#1904) Co-authored-by: Steve Herzog <vn54bh8@m-c02xf19ujgh7.homeoffice.wal-mart.com> * feature: open browser when proxy is ready (electrode-io#1905) Co-authored-by: Steve Herzog <vn54bh8@m-c02xf19ujgh7.homeoffice.wal-mart.com> * Fix broken reporter link (electrode-io#1908) * Update changelog * [Publish] - @xarc/app@11.0.1 - @xarc/app-dev@11.0.1 - @xarc/webpack@11.1.3 * Upgrade poc-subapp and poc-subapp-redux to react18 with rr6 (electrode-io#1913) * Upgrade POC-SUBAPP-REDUX to React 18 and RR6 * Upgrade poc-subapp to react 16 and RR6 * fix types issues in multiple packages (electrode-io#1918) * fix types issues in multiple packages Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> * fix: poc-subapp and merge changes from master * fix: update devtool to use eval-cheap-module-source-map * chore: remove debug statement Co-authored-by: Steve Herzog <srherzog@gmail.com> Co-authored-by: Steve Herzog <vn54bh8@m-c02xf19ujgh7.homeoffice.wal-mart.com> Co-authored-by: Ashu Verma <ashuverma@outlook.com> Co-authored-by: smuthya <srinath.reddy.muthyala@walmart.com> Co-authored-by: mristic505 <mateja.ristic@gmail.com> Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> * fix poc subapp redux when running with rushx (electrode-io#1920) * Upgrade poc-subapp and poc-subapp-redux to react18 with rr6 (electrode-io#1913) * Upgrade POC-SUBAPP-REDUX to React 18 and RR6 * Upgrade poc-subapp to react 16 and RR6 * fix types issues in multiple packages (electrode-io#1918) * fix types issues in multiple packages Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> * fix: sample app - poc-subapp-redux * chore: pnpm-lock updated * fix: incorrect filename reference in the import statement * fix: incorrect filename reference in the import statement Co-authored-by: mristic505 <mateja.ristic@gmail.com> Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> * feat: add commitlint configs (electrode-io#1921) * docs: documentation updates (electrode-io#1922) * docs: update readme and contribution guidelines * docs: documentation and contribution guidelines updated * docs: update release guidelines * docs: update release guidelines * chore: testing package publish flow (electrode-io#1925) * fix: update message log * fix: update message log * fix: update message log * cleanup (electrode-io#1927) * fix: publish flow * fix: clean up * rush test publish package (electrode-io#1928) * feat: message update - test rush publish * feat: message update - test rush publish * chore: bump versions [skip ci] (electrode-io#1929) * developer flow - change file creation - test (electrode-io#1930) * feat: message update - test rush publish * chore: rush change commit * chore: delete test packages created to test publish flow with rush (electrode-io#1931) * docs: update package publishing process with rush (electrode-io#1933) * Publishtest (#7) * feat: add a new test package for publish test * chore: new package to test publish operation - commiting change files * chore: bump versions [skip ci] * docs: update package publishing process with rush * chore: remove unused files * chore: remove unused files * chore: update shrinkwrap file * docs: update documentation with commitlint details (electrode-io#1934) * docs: update documentation with commitlint details * feat: rebase rush changes with master * docs: update documentations * fix: webpack version inconsistency issues * chore: enable temporarly commented lines in rush branch * chore: uncommenting a test case and un necessary console.log * chore: use rever to use clap keep minimal changes against master branch * chore: use mocha config same as master branch * chore: changelog jsons from rush change * fix: types error in xarc-app-dev package (#8) * fix: types error in xarc-app-dev package * docs: update to contribution guidelines * docs: update to contribution guidelines * fix: eslint issues in sample apps (#9) * fix: test issues in sample apps (#10) * fix: test issues in sample apps * fix: test issues in sample apps * chore: allow warnings in successful builds (#11) * fix: test issues in sample apps * fix: test issues in sample apps * chore: allow warnings in successful builds * chore: allow warnings in successful builds * fix: tests for node 12 (#12) * fix: test issues in sample apps * fix: test issues in sample apps * fix: tests for node 12 * test: fix unit test failing in subapp-web@2 * chore: update pnpm lock file * chore: add ci step to verify changelogs available * chore: add ci step to verify changelogs available * chore: add ci step to verify changelogs available --------- Co-authored-by: Ashu Verma <ashuverma@outlook.com> Co-authored-by: Srinath Muthyala <59977748+smuthya@users.noreply.github.com> Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com> Co-authored-by: mristic505 <mateja.ristic@gmail.com> Co-authored-by: Arun Vishnu <arunnampoothiri.vish@walmart.com> Co-authored-by: Steve Herzog <srherzog@gmail.com> Co-authored-by: Steve Herzog <vn54bh8@m-c02xf19ujgh7.homeoffice.wal-mart.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.