Kapil Bansal comments

Results 112 comments of


                                            Kapil Bansal

[refactor] Use utility function for calling popen() properly

The current problem I am facing with this is that if we create a function in `monitoring_utils.lua` and load it, then the io defined in first table, `TestNeighbor` or `TestInterface`...

Bugfix: Add duplicate organization name check

Hi @NenadPantelic are you still working on this?

Improvement: refactor expected_fields_validator

@KushalBeniwal comment once you do the setup task

Migrating Travis to Github action

Seems like an easy issue. Should this be marked for OSH as well?

Spike: Evaluate possible approaches on dealing with authentication

@mtreacy002 can this issue be a good one for `Open Source Hack`

Add tests to follow PEP8 guidelines

@Aryamanz29 are you working on this?

Add tests to follow PEP8 guidelines

Hi @mtreacy002 black is used to lint the code so to increase readability but flake8 ensures that the code follows Python standards specified by PEP8. There are some cases where...

Add tests to follow PEP8 guidelines

Yes, it looks good to. I was unaware of this. I think we can proceed with this instead of having flake8 and flask.

chores: Correct Grammatical Mistakes

Hi @unnati914 I made some updates in this PR and the comment as needed.

[WIP] Issue3 workflow doc

Hi @rpattath there is already a PR opened for [pull request workflow](#50 )