Skip to content

chore: Refactored code location for iterators#225

Merged
jonaslagoni merged 3 commits intoasyncapi:masterfrom
jonaslagoni:feature/move_iterators
Jan 15, 2021
Merged

chore: Refactored code location for iterators#225
jonaslagoni merged 3 commits intoasyncapi:masterfrom
jonaslagoni:feature/move_iterators

Conversation

@jonaslagoni
Copy link
Member

@jonaslagoni jonaslagoni commented Jan 15, 2021

Description
This PR moves iterators and constants to move towards a model which only contains the model code and not excess functionality to reduce complexity. This is a pre-step for me to move assignment of anonymous ids to schemas in #223

@jonaslagoni
Copy link
Member Author

Let me know if we should move the files into another directory or if its fine where it is

@jonaslagoni
Copy link
Member Author

Also, do we need new unit tests for these functions? 🤔

@jonaslagoni jonaslagoni merged commit 4ddc3ca into asyncapi:master Jan 15, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jonaslagoni jonaslagoni deleted the feature/move_iterators branch January 15, 2021 15:34
jonastg added a commit to jonastg/parser-js that referenced this pull request Jan 18, 2021
* master: (24 commits)
  chore: Refactored code location for iterators (asyncapi#225)
  chore(release): 1.3.1 (asyncapi#222)
  fix: apply traits to standalone messages from components section (asyncapi#214)
  test: improve feedback loop from browser tests (asyncapi#216)
  ci: fix the space-issue in bump workflow (asyncapi#218)
  ci: update global workflows (asyncapi#217)
  chore(deps): bump ini from 1.3.5 to 1.3.7 (asyncapi#215)
  ci: rename pr testing job name and test node 14 (asyncapi#213)
  ci: bump workflow to start on push instead of release (asyncapi#212)
  chore(release): 1.3.0 (asyncapi#211)
  feat: add a traverse schema function (asyncapi#198)
  ci: add workflow that bumps parser in other asyncapi repos (asyncapi#208)
  ci: fix release workflow step that is responsible for handling twitter (asyncapi#209)
  ci: update global workflows (asyncapi#206)
  ci: disable any testing on draft PR (asyncapi#204)
  chore(release): 1.2.0 (asyncapi#202)
  feat: extend the components and asyncapi model with has-like functions (asyncapi#192)
  chore(deps-dev): bump semantic-release from 17.0.6 to 17.2.3 (asyncapi#199)
  chore(release): 1.1.1 (asyncapi#197)
  fix: channels with name '/' fail on validation (asyncapi#196)
  ...
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 1.3.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants