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

Fix logic in where builds push to DHE #4

Merged
merged 1 commit into from
Sep 18, 2017
Merged

Fix logic in where builds push to DHE #4

merged 1 commit into from
Sep 18, 2017

Conversation

ewhoch
Copy link
Contributor

@ewhoch ewhoch commented Sep 18, 2017

Previously the logic was

isRelease -> Release
isContinuous -> Nightly

really it should be

isRelease -> Nightly

and the isContinuous part can just go away. Still requires the property set "published.package.release"

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

cbridgha
cbridgha approved these changes Sep 18, 2017
View reviewed changes
Copy link
Member

@cbridgha cbridgha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@cbridgha cbridgha merged commit f719828 into OpenLiberty:integration Sep 18, 2017
navidsh pushed a commit to navidsh/open-liberty that referenced this pull request Jan 17, 2018
@arturdzm
Merge pull request OpenLiberty#4 from navidsh/info-reader
d04297c 
Changed the annotation reader logic for Info annotation
anjumfatima90 pushed a commit to anjumfatima90/open-liberty that referenced this pull request Feb 7, 2018
@tjwatson
Merge pull request OpenLiberty#4 from tjwatson/indexDependencies
c6fa708 
Index dependencies
AidanPolese pushed a commit to AidanPolese/open-liberty that referenced this pull request Sep 24, 2018
@cbridgha
Merge pull request OpenLiberty#4 from gmarcy/more-kernel-test-projects
239de9a 
More kernel test projects
@Emily-Jiang Emily-Jiang mentioned this pull request Feb 25, 2019
Closed
herman-kailey pushed a commit to herman-kailey/open-liberty that referenced this pull request Sep 26, 2019
@TayyabDev
Merge pull request OpenLiberty#4 from herman-kailey/mavenRepo
8df82c7 
removing lambda in ArtifactDownloaderUtils
volosied pushed a commit to volosied/open-liberty that referenced this pull request Mar 23, 2020
@cbridgha @volosied
# This is a combination of 5 commits.
82a576f 
# This is the 1st commit message:

Setup transform task

Enable transform in build

add el-4.0 jakarta feature and api

add jsp 3.0 feature

change servlet 5 symbolic bundle name to jakartaee; update servlet-5.0/resources/l10n descriptions

package & rule corrections

update kind=noship

Update transformer version

rename fixes & add jsp resources

update jsp 3.0 bnd.bnd buildpath

Add Jak 9 feature and upgrade transformer

Setup transform task

Enable transform in build

add el-4.0 jakarta feature and api

add jsp 3.0 feature

change servlet 5 symbolic bundle name to jakartaee; update servlet-5.0/resources/l10n descriptions

package & rule corrections

update kind=noship

Update transformer version

rename fixes & add jsp resources

# This is the commit message OpenLiberty#2:

update jsp 3.0 bnd.bnd buildpath

# This is the commit message OpenLiberty#3:

RemoveExtra feature file

# This is the commit message OpenLiberty#4:

update jakarta servlet version rule; correct servlet5  eclipse package name

# This is the commit message OpenLiberty#5:

delete jakarta jar before transformatiton
brenthdaniel added a commit to brenthdaniel/open-liberty that referenced this pull request Jul 11, 2022
@brenthdaniel
Merge pull request OpenLiberty#4 from brenthdaniel/master
e661e29 
Update README
tevans78 pushed a commit that referenced this pull request Jul 13, 2022
@yasmin-aumeeruddy
Mp telemetry (#4)
047c255 
* Telemetry feature added
volosied added a commit to volosied/open-liberty that referenced this pull request Aug 16, 2022
@volosied
# This is a combination of 14 commits.
f8aceeb 
# This is the 1st commit message:

drop 3.1 code

# This is the commit message OpenLiberty#2:

update jsp.3.1 to jakarta

# This is the commit message OpenLiberty#3:

enable failing el tests

# This is the commit message OpenLiberty#4:

Update spec level for 3.1 bundle

# This is the commit message OpenLiberty#5:

Fix transformations with new 3.1 bundle

# This is the commit message OpenLiberty#6:

Working pages 3.1 feature

# This is the commit message OpenLiberty#7:

groudwork for errorOnELNotFound

# This is the commit message OpenLiberty#8:

add .project

# This is the commit message OpenLiberty#9:

create page directive errorOnELNotFound

# This is the commit message OpenLiberty#10:

create page directive errorOnELNotFound - 23.1 bundle

# This is the commit message OpenLiberty#11:

WIP: jsp prop group for erroronelnotfound

# This is the commit message OpenLiberty#12:

Fix jsp 3.1 bnd

# This is the commit message OpenLiberty#13:

Fix jsp prop group finally

# This is the commit message OpenLiberty#14:

Almost workking jsp.factories ( need to fix getErrorOnELNotFound ) in interface
@pnicolucci pnicolucci mentioned this pull request Jan 5, 2024
Merged
benjamin-confino referenced this pull request in benjamin-confino/open-liberty Jan 29, 2024
@yasmin-aumeeruddy
Merge pull request #4 from benjamin-confino/yasmin-telemetry-metrics
cec68a9 
refactor to remove 2.0.common
@jhanders34 jhanders34 mentioned this pull request Apr 23, 2024
Merged
BerksanAtes added a commit to BerksanAtes/open-liberty that referenced this pull request Jun 18, 2024
@BerksanAtes
# This is a combination of 4 commits.
68471f1 
# This is the 1st commit message:

CXF Original

# This is the commit message OpenLiberty#2:

Traces are added

# This is the commit message OpenLiberty#3:

Remove FAT bundle changes

# This is the commit message OpenLiberty#4:

WIP
utle pushed a commit to utle/open-liberty that referenced this pull request Oct 10, 2024
@mshah0722
Merge pull request OpenLiberty#4 from utle/FIPSwork
d15c6d4 
add common crypto to ssl feature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbridgha cbridgha cbridgha approved these changes

Assignees
No one assigned
Labels
CLA Not Needed release-17.0.0.4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ewhoch @LibbyBot @cbridgha