-
Notifications
You must be signed in to change notification settings - Fork 157
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.
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
Move classes from PowerShell to C# #350
Changes from 1 commit
aeeeb6b
e8df17c
5930f82
f79fd6f
f2ece6b
b6674c2
05f8130
91b43ad
23cf953
c58028d
1f7e0b3
140ee72
627ca4f
5fb4666
97bdcb9
aede2ed
2d8b7a7
8f11a02
8a5813b
9396b89
71ec7cd
162530a
faccf1d
c0e83f4
0c1b76d
a363c0c
05dd357
1cc638a
0df9e68
fe37628
cf628b9
ab10f4f
63af4b0
9bef391
a9f6223
976510b
55c6e80
85f38e4
42286aa
f04c60b
a0be02a
8d88b43
6946364
fc385b0
50ede9c
c76ace3
ce5d758
1e44464
f8bfb74
75ef0e6
c0d8b14
8bae394
f59c7aa
ae9eae4
52d8a1d
a2d1797
75051fe
66e35d6
0a5a37d
cc31c47
58f5012
8379155
9c95363
7c92d63
01a6289
50afb46
dc33d38
1d59060
2ff6dae
1dcf24a
c4bc4ff
554c8d8
ce3481e
4fe112e
6a8d105
e0d6d6f
d6e736e
8fb4637
bfcdca5
f5dd8dc
3d8de33
6673fb0
33befe3
350ec14
cb83d21
45e1103
96e3ec2
72d24c0
34509f1
4c89dc4
59af24b
da8b90b
d01c0be
017b6d9
727efcc
a42ae3b
ad22334
0ad42be
3572034
de1247b
6d4658e
e82daa1
9b4905a
ca3f16c
843bedc
f780d71
6f599e1
339338b
61ebca7
1ddd93a
ce9d6f5
c4001d3
3a9d89d
db9798b
be40b99
2629bd6
035bc8b
a974e87
50d945f
d916615
6fad140
d988d90
a2709f0
348ef97
fd6dd26
f3b1e89
1c7f749
8433d22
f27e31f
6c4e73c
4c93164
88283fb
9289505
f24a065
526e1fb
490cc94
9b3bf5c
17709d1
5c965ba
c2e853f
2e583b7
30ab898
2ab994d
dea1a85
c5ff8af
60c8557
10365bc
8bb1c27
a54eb45
0717221
3b0a65a
9eb70f7
4511d6e
7a8c461
17ec1b9
71a0fcd
d74954a
3507410
245a032
9dc6405
0eed7d5
bb905b5
0682a09
4691d51
c444f58
adc627c
2a5a35d
80fd54f
25364f4
4d4120e
8e5e6ed
3960c64
c5fcfa3
3dea581
35d09dd
0e7738d
8318244
2f220bf
c7bd2b1
1a35d15
9401d91
4b4c1c3
3ae1701
0ad43e6
75d364b
a3a20d0
8368756
55f7654
aac015e
3ec298f
5f3e24f
1adb5bc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,7 +8,7 @@ | |
# https://bit.ly/Add-VSTeamBuildDefinition | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is this witchcraft? What did you do there or how? That is cool. Did you add every cmdlet here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As I reviewed each function I found the docs page and created the short link. I don't have one for each yet but I will continue to create as I review the functions. |
||
|
||
function Add-VSTeamBuildDefinition { | ||
[CmdletBinding()] | ||
[CmdletBinding(HelpUri='https://methodsandpractices.github.io/vsteam-docs/modules/vsteam/Add-VSTeamBuildDefinition')] | ||
param( | ||
[Parameter(Mandatory = $true, ValueFromPipelineByPropertyName = $true)] | ||
[string] $InFile, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DarqueWarrior I hope you don't kill me, but due to technical improvements I had to change the basic url from
https://methodsandpractices.github.io/vsteam-docs/modules/vsteam/
to
https://methodsandpractices.github.io/vsteam-docs/docs/modules/vsteam/
there is the "docs" path being added. Nothing will change in the future. not sure if I can get rid of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem that is an easy change with regular expressions. To confirm I just need to add 'docs' between 'vsteam-docs/' and '/modules/". Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes confirmed!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When will a new version of the site be pushed with the path change? I just noticed the new paths are 404s right now. I would like this to be updated again before I merge the latest changes into master.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, @SebastianSchuetze any ETA on when a new version of the docs site will be pushed so the new URLs like this one works?
https://methodsandpractices.github.io/vsteam-docs/docs/modules/vsteam/Add-VSTeam
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can push it today in 1 hour. Just tell me soon after if we keep one module or split in two. I prepared sub doc pages for vsteam plus, but it is basically just menu and placeholder intro pages.