Comments (8)
Question @aRTy42 what item types are there that don't have implicit mods unless corrupted? Only jewels? This is the problem:
; Retrieve items implicit mod if it has one
If (Item.IsWeapon or Item.IsArmour or Item.IsRing or Item.IsBelt or Item.IsAmulet) {
from poe-iteminfo.
Should be fixed (unless there are more item types that aren't included).
from poe-iteminfo.
Yes, I think only jewels are missing there.
from poe-iteminfo.
Btw, I'm currently improving the update function to allow :
x.x.x-alpha.x
x.x.x-beta.x
x.x.x-rc.x
Where rc
> beta
> alpha
, should a release candidate be higher/newer than a beta?
from poe-iteminfo.
I think that is fine, but I have practically no expertise in that regard.
from poe-iteminfo.
I don't have that, too, just took a quick look at semver.org but looking at bit more I just found this:
A larger set of pre-release fields has a higher precedence than a smaller set, if all of the preceding identifiers are equal. Example: 1.0.0-alpha < 1.0.0-alpha.1 < 1.0.0-alpha.beta < 1.0.0-beta < 1.0.0-beta.2 < 1.0.0-beta.11 < 1.0.0-rc.1 < 1.0.0.
Btw, according to semantic versioning you're doing it wrong, there shouldn't be a leading zero anywhere 2.5.05
;-)
And you're incrementing the patch version too much while the minor version not often enough
MAJOR version when you make incompatible API changes, MINOR version when you add functionality in a backwards-compatible manner, and PATCH version when you make backwards-compatible bug fixes.
Not that I care much about it though.
from poe-iteminfo.
I'm using the version number to indicate for/during which PoE version it was released. PoE 2.5 gets the 2.5.x versions.
Changing the version number depending on the kind of update is a reasonable thing, but judging from what I read are people updating their script so differently and randomly that I doubt that it makes any difference. In particular now with the update notifier people will likely not look for the version number and decide based on that whether to update or not. Also semver.org is about software with an API, so the whole point of the strict definitions when to increase which version number part is done to deal with API/software conflicts. This seems not really applicable for ItemInfo.
I was using a leading zero because sorting seems more reliable that way, as some systems/environments use 1 < 11...19 < 2 < 21...29 < 3 etc. The updater was fine with it, wasn't it? If it is only a recommendation by the GitHub cofounder guy there, I'd rather stick to leading zeros.
from poe-iteminfo.
I think its less about having an API or not.
The updater doesn't have a problem with it and if it does it's easy to remove that leading zero for comparison purposes. There's no good reason to change your versioning logic aside from using a more standard approach. I actually didn't know that yours is related to the PoE version. Emmit does the same and it's not a bad idea in general.
The usual method I have seen is X.Y.Z, which generally corresponds to major.minor.patch:
Major version numbers change whenever there is some significant change being introduced. For example, a large or potentially backward-incompatible change to a software package.
Minor version numbers change when a new, minor feature is introduced or when a set of smaller features is rolled out.
Patch numbers change when a new build of the software is released to customers. This is normally for small bug-fixes or the like.
I actually made a breaking change to the macro once without realising it and changing the version accordingly, it was when we renamed a lot of files/folders and I have a few datafiles urls hardcoded to download them from the repo. Older versions didn't work anymore.
from poe-iteminfo.
Related Issues (20)
- 8mod map identified as 9mod HOT 3
- New Item Mods - unknown mod HOT 10
- Search Opal Ring with ilvl by default HOT 1
- SendCtrlC HOT 4
- Error : target label does not exist HOT 10
- to support French
- Don't remove clipboard contents HOT 1
- Highlight items gets old values HOT 1
- Error At Line 5 HOT 1
- Can't create control error HOT 2
- [Feature Request] Update MapList.txt for 3.6.0 HOT 1
- Tooltip timeout doesn't work
- is this normal? HOT 2
- Add hideout information to maps
- Display at fixed coordinates with MouseGetPos
- ItemInfo reports disscontinued stats from wiki (Queen's_Escape) HOT 2
- Currencyinfo from poe.ninja not available at startup HOT 1
- master
- No rare estimate HOT 1
- Script run but nothing work in game HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from poe-iteminfo.