Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fileutil logging #198

Open
wants to merge 227 commits into
base: master
Choose a base branch
from
Open

Fileutil logging #198

wants to merge 227 commits into from

Conversation

plty
Copy link

@plty plty commented Nov 12, 2018

No description provided.

plty and others added 30 commits September 5, 2018 01:50
@plty
change repo badges source from se-edu into our organization
f92884a
Merge pull request #3 from CS2103-AY1819S1-W12-1/readme-edit
c07e8c6 
change repo badges source from se-edu into our organization
@plty
add encrypted slack notification token
b6e89c7
@plty
reformat travis.yml file
7d6b4f8
@plty
add unencrypted slack notification token
4a79844
@plty
remove team-members placeholder images
04f671e
@plty
add team-members placeholder photos
94a1cd5
@plty
change AboutUs.adoc template
8961fed
Merge pull request #4 from CS2103-AY1819S1-W12-1/travis-ci-slack-inte…
50f4fc0 
…gration

Travis CI Slack Integration
@plty
Merge branch 'master' of github.com:nus-cs2103-AY1819S1/addressbook-l…
8f994d6 
…evel4 into dev
@plty
Merge branch 'master' into dev
095b985
@plty
move coveralls link from address-book to main
0b884f0
@plty
Merge branch 'coveralls-fix' into dev
faa7e0f
@plty
move appveyor link from address-book to main
54c072b
Merge pull request #8 from CS2103-AY1819S1-W12-1/dev
e4eb9d5 
repo sync with upstream, coverrals and appveyor fix
Update badge in README to new repo name
c03d391
@plty
remove old codacy badge
bbb044b
Update codacy source
8e8d139
Update badge link
c6e3257
@plty
change travis badge image source
4ba73e3
@plty
update wrong travis image source change
21a7388
Merge pull request #11 from CS2103-AY1819S1-W12-1/readme/badge
41c4857 
Update badge in README to new repo name
@plty
Merge pull request #12 from nus-cs2103-AY1819S1/master
0b5eedb 
DeveloperGuide: update "Setting up" section
@plty
add swiftkage and dan-delion photo and template for kevvvinn
7aae71d
@plty
moving the photo link to the respective file
de1dfff
@plty
Merge branch 'documentation' of github.com:plty/nuspro-lost-and-found…
6e646f2 
… into documentation
@plty
remove learning outcomes LearningOutcomes.adoc
224d80c
@plty
amend readme, change readme into lost and found
b0a07e8
@plty
User Guide Changes
a1cf8f6
@plty
change developer guide
ee1511b
plty and others added 28 commits November 7, 2018 23:09
@plty
i have to go home
ca8c200
@plty
fix xml test
87b3889
@Swiftkage
Merge pull request #18 from CS2103-AY1819S1-W12-1/master
bb41052 
Update from upstream
@Swiftkage
Update DG and test cases
1889cf8
@Swiftkage
Upate project specs
eba398c
@Swiftkage
Update UI
98445d1
@Swiftkage
Merge pull request #129 from Swiftkage/master
9631c6b 
Update project specs
@Swiftkage
Update test case data
2c50385
@Swiftkage
Change Article Identity fields
a472fa0 
And also relevant test cases
@Swiftkage
Update DG
7e0d4ca
@Swiftkage
Update DG
40ff8a3
@Swiftkage
Update Swiftkage PPP
a524b27
@Swiftkage
Merge pull request #130 from Swiftkage/master
800bf10 
Update DG and PPP
@plty
xml parser bug fix by setting image to optional
0f07b38
@Swiftkage
Update DG, PPP and find-t
50510f3
@plty
checktyle related updates
c77d207
@Swiftkage
Update test cases
eb60eb1 
Added test data for resolved articles.
Made sure filteredList test are working.
@Swiftkage
Merge pull request #131 from Swiftkage/master
5c91812 
Update DG, PPP and find-t
@Swiftkage
Add test cases for Edit
364c4ca
@Swiftkage
Fix checkstyle
ac2c6a9
@Swiftkage
Update Add and Edit test cases
8ff76c5
@plty
merge with master
66d4f07
@plty
resolve conflict with newest merge
060b639
@plty
Merge branch 'image-integration'
7a73093
@plty
Merge pull request #132 from Swiftkage/master
6959b27 
Update test cases
@plty
first draft
adb87a0
@plty
jerrell PPP
9afeb3e
@plty
improved logging
9c0efb8
@nus-se-pr-bot
Copy link

Hi @plty, your pull request title is invalid.

For PR sent to satisfy a learning outcome, the PR name should be in the format of [Learning Outcome ID][Team ID] Your Name, where [Learning Outcome ID] has no dashes or spaces (e.g. [W3.1a]) and [Team ID] has one dash only and no spaces (e.g. [W14-2] means Wednesday 2pm (14 hrs), Team 2).
E.g. If you are in tutorial W09 (i.e. Wednesday 9am), team 1, and is submitting a PR for LO W2.2b, the PR title would be [W2.2b][W09-1] James Yong. Note that your tutorial IDs are different from those shown in CORS/IVLE.

For team PR, the PR name should be in the format of [Team ID] Product Name (e.g. [T09-2] Contact List Pro).

Please follow the above format strictly and edit your title for reprocessing.

Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at nus-se-pr-bot and add a link to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
FormatCheckRequested
Projects
None yet
Milestone
No milestone
3 participants
@plty @nus-se-pr-bot @Swiftkage