frontera icon indicating copy to clipboard operation
frontera copied to clipboard

I have done some workaround across implementing ActiveMq to the messagebus so would like to contribute

Open amitsing89 opened this issue 7 years ago • 6 comments

Hi,

I have started once to integrate the activeMq in the messagebus so i would like to add that to the repository, so if you can allow kme to contribute then it would be helpful to add a new component

amitsing89 avatar Oct 24 '18 07:10 amitsing89

Codecov Report

:exclamation: No coverage uploaded for pull request base (exception-dbw@dec0e02). Click here to learn what that means. The diff coverage is 57.89%.

Impacted file tree graph

@@               Coverage Diff               @@
##             exception-dbw    #350   +/-   ##
===============================================
  Coverage                 ?   60.3%           
===============================================
  Files                    ?      78           
  Lines                    ?    6145           
  Branches                 ?     809           
===============================================
  Hits                     ?    3706           
  Misses                   ?    2264           
  Partials                 ?     175
Impacted Files Coverage Δ
frontera/utils/twisted_helpers.py 78% <ø> (ø)
frontera/utils/graphs/data.py 81.81% <ø> (ø)
frontera/contrib/backends/remote/messagebus.py 80% <0%> (ø)
frontera/contrib/scrapy/messagebus_stats.py 0% <0%> (ø)
frontera/utils/s3.py 0% <0%> (ø)
frontera/strategy/depth.py 0% <0%> (ø)
frontera/strategy/discovery/__init__.py 0% <0%> (ø)
frontera/contrib/scrapy/schedulers/recording.py 0% <0%> (ø)
frontera/strategy/discovery/sitemap.py 0% <0%> (ø)
...ontera/contrib/messagebus/kafka/offsets_fetcher.py 14.89% <0%> (ø)
... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update dec0e02...84f9e10. Read the comment docs.

codecov[bot] avatar Nov 15 '18 08:11 codecov[bot]

Hey @amitsing89 , I think you should rebase it to latest master, it seems to me your code is based on some outdated version.

sibiryakov avatar Nov 15 '18 08:11 sibiryakov

Currently, it's extremely hard to review because of enormous amount of unrelated changes.

sibiryakov avatar Nov 15 '18 09:11 sibiryakov

i will update it and will try to start it again

amitsing89 avatar Nov 15 '18 12:11 amitsing89

@amitsing89 You should rebase your changes on top of the latest master as suggested. Have you used Git rebase before? Have you added the main repository as an upstream remote in your local clone of your fork?

Gallaecio avatar Apr 11 '19 13:04 Gallaecio

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.