Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

handle comments#735

Merged
boydc2014 merged 3 commits intodonglei/lu-refactorfrom
liweitian/lu-refactor2
Aug 22, 2019
Merged

handle comments#735
boydc2014 merged 3 commits intodonglei/lu-refactorfrom
liweitian/lu-refactor2

Conversation

@liweitian
Copy link
Contributor

Description

modify test cases and improve code readability

Task Item

Type of change

Please delete options that are not relevant.

  • Code refactor (non-breaking change which improve code quality, clean up, add tests, etc)

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have functionally tested my change

Screenshots

Please include screenshots or gifs if your PR include UX changes.

@liweitian liweitian added the Not Targetting Main a pull request created not against main label Aug 22, 2019
@boydc2014 boydc2014 merged commit 1891bb2 into donglei/lu-refactor Aug 22, 2019
@boydc2014 boydc2014 deleted the liweitian/lu-refactor2 branch August 22, 2019 11:07
boydc2014 added a commit that referenced this pull request Aug 23, 2019
* rename luPath to botDir

* Move things from luIndexer to luPublisher

* remove unused files interfaces

* clean

* clean up

* remove invalid interface

* update publish time after publish, instead of before

* save after publish

* move the length checking at later stage and rename some variables

* simple test case is better than skiped tests

* fix bugs and test cases and no longer add luis.status.json into this.files (#730)

* fix some typos & grammar

* don't muate the objects

* handle comments (#735)

* handle comments

* fix test case

* fix test cases
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Not Targetting Main a pull request created not against main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants